-
Notifications
You must be signed in to change notification settings - Fork 30
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
New Feature: Show Toolbars Only When The Mouse Hovered In #42
Comments
thank you for that idea.. could you explain what we try to achieve with this feature? |
We could pin some screenshots on the screen. |
okay, so an image viewer-like functionality? isn't that what image viewers like imv are for? |
While this is true, we can add annotations to the image. |
okay.. well I think there is nothing wrong with hiding the toolbars on demand. I would prefer a shortcut for this, like I must say that I am a big fan of explicit behaviour (shortcut/button toggles toolbar) and not dynamic behaviour (mouse hover toggles toolbar). Further I think that should be "opt-in" .. not the default behaviour. There might be a setting "hide-toolbar-on-startup" or something similar.. I am happy to accept PRs |
I implemented the basic functionality #45 , but there isn't support for configuration. |
Now, it supports configuration 6fb3d68. |
I think most people find this very convenient, but for users who frequently use keyboard shortcuts (such as Window Manager, Vim-like Editor, Emacs), it is not convenient. |
You're right, Plus it's will increase development time which would be wasteful as this dosen't offer much as keyboard based implementation will still be there plus the hovering toolbar. |
how does it work when you want to edit the fullscreen image, e.g. without crop? |
It's been ages since I took fullscreen shot. Using |
the first basic functionality has been released as https://github.com/gabm/Satty/releases/tag/v0.10.0 |
The toolbars don't display if the mouse doesn't hover in the app window.
And show the toolbars when we hover in.
The text was updated successfully, but these errors were encountered: