-
Notifications
You must be signed in to change notification settings - Fork 40
Add hotkeys, incl. macOS + Windows global hotkeys #31
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
Conversation
Updated hotkeys taken from this PR: flameshot-org/flameshot#1097 Ref: flameshot-org/flameshot#1593
If you want use Flameshot as a default screenshot utility, chances are you want to launch it using the <kbd>Prt Sc</kbd> key. Flameshot doesn't yet offer a fully-automated option to do so, but you can configure your system to do so. | ||
|
||
Flameshot uses <kbd>Print screen</kbd> (Windows) and <kbd>cmd</kbd>-<kbd>option</kbd>-<kbd>shift</kbd>-<kbd>4</kbd> (macOS) as default global hotkeys. | ||
|
||
On Linux, Flameshot doesn't yet support <kbd>Pr Scr</kbd> out of the box, but with a bit of configuration you can set this up: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar comment to the #1845
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replied over there: flameshot-org/flameshot#1845 (comment)
| <kbd>Print screen</kbd> | Capture Screen | | ||
| <kbd>Shift</kbd> + <kbd>Print</kbd> | Screenshot History | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Considering that these, unlike others, are not for the GUI mode, and considering that they are not universal (e.g Mac does not even have PrntScrn button), I suggest removing these.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are changes that I took from @DbDibyendu's PR flameshot-org/flameshot#1097 (merged by @borgmanJeremy), so if there should be any changes done here, then they should probably also be done to the readme in https://github.com/flameshot-org/flameshot
@karlhorky Can you solve the conflicts? |
@ZetaoYang opened #86 for this, closing this PR. |
* Add hotkeys, incl. macOS + Windows global hotkeys Updated hotkeys taken from this PR: https://github.com/flameshot-org/flameshot#1097 Ref: https://github.com/flameshot-org/flameshot#1593 Ref: #31 * Lowercase keys, add note about defaults
Updated hotkeys taken from this PR: flameshot-org/flameshot#1097
Ref: flameshot-org/flameshot#1593