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

[Feature request]: Allow Esc to close Calendar #62

Closed
pkundrat opened this issue Sep 13, 2016 · 6 comments
Closed

[Feature request]: Allow Esc to close Calendar #62

pkundrat opened this issue Sep 13, 2016 · 6 comments

Comments

@pkundrat
Copy link

T-Clock calendar is a great way to show calendar week numbers, but there is no quick way to close it.
Mouse works, Alt-F4 works too - but Esc is way faster than either of those and is also natural key for this.
I would do the fix myself, but setting up the whole environment and getting to know the code structure would take couple of days, so i am just hoping current developers could implement this in couple of minutes.

@stdedos
Copy link

stdedos commented Nov 13, 2016

I feel the could be an option to make it act like the Windows Calendar too: If you click anywhere outside of the calendar, it just dies

@White-Tiger
Copy link
Owner

White-Tiger commented Nov 13, 2016

If you click anywhere outside of the calendar, it just dies

that's already there... called: Close calendar on lose focus (which should be default)

@pkundrat
Copy link
Author

pkundrat commented Nov 17, 2016

Thanks - thats helpful, but not exactly as useful. It is sufficient for
quick looks into the calendar.
But if you want to see the calendar while working with some other program -
you dont want to close on lost focus.
So i believe Esc still makes sense (and is much faster than mouse).
I wouldnt rate is as important, only cosmetic- having Esc is also valuable
from UI consistency POV - Esc to close is quite common for transient popups
(similar to X button).
You could live without X button too, but you would miss it.

@White-Tiger
Copy link
Owner

I never said anything about not adding it :P In fact, it's working in my local version....

@pkundrat
Copy link
Author

pkundrat commented Nov 19, 2016

Haha - got me there :) Thumbs up :)

@White-Tiger
Copy link
Owner

sorry... it took a while before I've released it, but here you go ;)
v2.4.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants