cdown / clipmenu

Clipboard management using dmenu
MIT License
1.14k stars 91 forks source link

store: use fixed length hex hashes for dir names #239

Closed N-R-K closed 1 day ago

N-R-K commented 1 week ago

mostly cosmetics. having fixed width dirnames results in a nicer listing of the directory. it also matches usual expectation of hashes being stringified in hex format.

image

cdown commented 1 week ago

I'm not sure I understand the justification, could you go over it more?

N-R-K commented 1 week ago

With the current naming, directories would look something like this:

image

With fixed-width hex naming, they all have the same width and appear nicer when listing (or when viewed thru a file manager etc).

It's entirely a cosmetic/aesthetic thing. Figured I'd raise it before there's a stable version released in case you like it, otherwise I can just keep the patch locally if you don't.

cdown commented 1 day ago

I was on the fence because while it does make hashes more uniform, it also adds (ok, not much) more work and code. I do think it looks better and is also visually more obvious when seeing clip hashes in CM_DEBUG, though, so let's do it. Thanks!