-
Notifications
You must be signed in to change notification settings - Fork 71
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reviewing update 6.01 #20
Comments
Thanks for this full review, its very helpful.
cheers |
Looking forward to v 6.1.1.:-) BTW, on renaming the program to LibreCrypt, how are you going to rename the "boxes" as well as the used extension? I think the original name file/folder container was a suitable one and "box" was a rather unfortunate as it terminologically collided with memo boxes, Being an occasional amateur Delphi programmer, may I suggest a possible workaround how to automatically close the Windows format dialog box - searching for its caption name and force-closing app/applet that has it. This can be done quite easily, though this would not work with other language versions of Windows. When do you expect to release the next update? |
Yes the 'boxes' will be renamed to 'containers'. I don't know about the extension yet, any suggestions? |
I would stick to the original extension - .vol (=volume). I didn't really like the ".box"...:-) |
There is also one more bug that I noticed. When running DoxBox or DoxBox Explorer from a read-only medium (e.g., CD/DVD-ROM) an error box is invoked on the program's launch informing that the INI file is read-only and the settings cannot be saved. I would understand such a behavior if I attempted to change a setting, but I did not attempt to change any, realizing that it's all read-only. To prevent possible error massages I also used "/readonly" command line parameter. And as if this was not enough, the same error box fires once more - also when closing the program from the read-only location. All this is unnecessary and should be fine-tuned. The error box should fire only if changes have been made, NOT if they have not been made. |
Thanks for this. The program saves a value to the INI file to say it is running, this is used to detect crashes, which is why you get the error message on every launch. |
All these issues should be fixed in 6.2 Beta, except the format dialog. |
Given 6.2 is out closing this ticket. |
Well done with the new color interface and all the changes and bug fixes made! It's good to see this great prog alive and kicking...:-)
Here are also some observations and tips for further improvement:
The text was updated successfully, but these errors were encountered: