Skip to content
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

Capitalise hash output #177

Closed
EatDesk opened this issue Feb 16, 2021 · 7 comments
Closed

Capitalise hash output #177

EatDesk opened this issue Feb 16, 2021 · 7 comments
Labels

Comments

@EatDesk
Copy link

EatDesk commented Feb 16, 2021

Hello, it's just a small detail but could you start capitalising the checksums in the "Checksum information" output again? I believe this changed from upper to lower case in the most recent release. I find it's a lot more legible in upper case if you're quickly looking at checksums.

Hopefully this isn't a setting or something I missed. Thanks!

@mcmilk
Copy link
Owner

mcmilk commented Mar 17, 2021

okay ... we should add some gui element and make it an option... could you provide a pull request for this ?

@EatDesk
Copy link
Author

EatDesk commented Mar 17, 2021

okay ... we should add some gui element and make it an option... could you provide a pull request for this ?

It's already capitalised in normal 7zip and I think it used to be capitalised in 7zip zstandard, I was just asking if you could restore it, unless it's an issue on my side?

@mcmilk
Copy link
Owner

mcmilk commented Mar 17, 2021

Most programs use lower case, so someone suggested I should use it also.
I will try to make it an option in near future ...

@mcmilk mcmilk added the todo label Mar 19, 2021
@mcmilk mcmilk closed this as completed in f090495 Apr 6, 2021
@User198263321
Copy link

User198263321 commented Nov 8, 2021

Since it was uppercase in the original 7-zip, uppercase should be default in 7-zip-zstd, but an option for lowercase in the settings would be nice. I also suggested this but it's reverted now. #152

Maybe an option could be added even where you could have Uppercase/lowercase hash in the normal context menu or advanced context menu (holding shift) separately. You could either have it all lowercase, all uppercase, or lowercase in normal and uppercase in advanced, or uppercase in normal and lowercase in advanced. Or just put them both in the context menu as option!

It could be something like "7-Zip ZS Hash [UP-CASE]"
or "7-Zip ZS Hash [LOW-CASE]"

Or something more grammatically correct

@User198263321
Copy link

Or if this could work somehow, you hold shift when you select a hash in the context menu to calculate, and you can flip what the default is in settings (shift for caps or shift for lower).

@uxmy
Copy link

uxmy commented Mar 10, 2022

Upper case letters are more comfortable for eyes, and lead to less mistakes. Definitely this is a bad change in the program.

@mcmilk
Copy link
Owner

mcmilk commented Mar 10, 2022

It's an option ... just change it.

mcmilk added a commit that referenced this issue Feb 12, 2023
- see issue #177 for more info about this

Signed-off-by: Tino Reichardt <[email protected]>
mcmilk added a commit that referenced this issue Feb 12, 2023
Re-add clearing of various history tracking within registry
- see issue #113 for comments on this
- you can re-enable this via tools->options->settings

Re-add: selectable uppercase / lowercase hash formatting
- see issue #177 for more info about this

Signed-off-by: Tino Reichardt <[email protected]>
mcmilk added a commit that referenced this issue Feb 12, 2023
Re-add clearing of various history tracking within registry
- see issue #113 for comments on this
- you can re-enable this via tools->options->settings

Re-add: selectable uppercase / lowercase hash formatting
- see issue #177 for more info about this

Signed-off-by: Tino Reichardt <[email protected]>
mcmilk added a commit that referenced this issue Feb 12, 2023
Re-add clearing of various history tracking within registry
- see issue #113 for comments on this
- you can re-enable this via tools->options->settings

Re-add: selectable uppercase / lowercase hash formatting
- see issue #177 for more info about this

Signed-off-by: Tino Reichardt <[email protected]>
mcmilk added a commit that referenced this issue Feb 12, 2023
Re-add clearing of various history tracking within registry
- see issue #113 for comments on this
- you can re-enable this via tools->options->settings

Re-add: selectable uppercase / lowercase hash formatting
- see issue #177 for more info about this

Signed-off-by: Tino Reichardt <[email protected]>
mcmilk added a commit that referenced this issue Feb 12, 2023
Re-add clearing of various history tracking within registry
- see issue #113 for comments on this
- you can re-enable this via tools->options->settings

Re-add: selectable uppercase / lowercase hash formatting
- see issue #177 for more info about this

Signed-off-by: Tino Reichardt <[email protected]>
mcmilk added a commit that referenced this issue Feb 13, 2023
Re-add clearing of various history tracking within registry
- see issue #113 for comments on this
- you can re-enable this via tools->options->settings

Re-add: selectable uppercase / lowercase hash formatting
- see issue #177 for more info about this

Signed-off-by: Tino Reichardt <[email protected]>
mcmilk added a commit that referenced this issue Feb 13, 2023
Re-add clearing of various history tracking within registry
- see issue #113 for comments on this
- you can re-enable this via tools->options->settings

Re-add: selectable uppercase / lowercase hash formatting
- see issue #177 for more info about this

Signed-off-by: Tino Reichardt <[email protected]>
mcmilk added a commit that referenced this issue Feb 18, 2023
- the last fix was not done correctly
- issue #177 is solved really now

Signed-off-by: Tino Reichardt <[email protected]>
mcmilk added a commit that referenced this issue Feb 18, 2023
- the last fix was not done correctly
- issue #177 is solved really now

Signed-off-by: Tino Reichardt <[email protected]>
mcmilk added a commit that referenced this issue Feb 18, 2023
- the last fix was not done correctly
- issue #177 is solved really now

Signed-off-by: Tino Reichardt <[email protected]>
mcmilk added a commit that referenced this issue Feb 18, 2023
- the last fix was not done correctly
- issue #177 is solved really now

Signed-off-by: Tino Reichardt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants