waycrate / wayshot

Mirrored at https://git.sr.ht/~shinyzenith/wayshot | screenshot tool for wlroots based compositors implementing zwlr_screencopy_v1
https://crates.io/crates/wayshot
BSD 2-Clause "Simplified" License
108 stars 18 forks source link

feat: added --human-time flag #92

Closed rachancheet closed 4 months ago

rachancheet commented 4 months ago

--human-time flag uses more convenient 24-hour format rather than UNIX_EPOCH seconds when saving image to disk.

rachancheet commented 4 months ago

if you want date as well , Please let me know .

Shinyzenith commented 4 months ago

Thanks for the pr but I am mostly hesitant about adding new crates to the dependency graph. Is there some other options we can explore / features to disable other parts of introduced crate ( I don't have experience with the crate you are using and hence no knowledge of it either )

I like the feature however and the flag name could probably be better named for being intuitive. Also this needs to be documented if we wish to merge this change.

Shinyzenith commented 4 months ago

Can you rebase this on feature freeze andreas branch?