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

Capture hotkey - PrintScrn #18

Open
elbertb opened this issue Jul 29, 2020 · 8 comments
Open

Capture hotkey - PrintScrn #18

elbertb opened this issue Jul 29, 2020 · 8 comments

Comments

@elbertb
Copy link

elbertb commented Jul 29, 2020

Hi,

Can you make it possible to make the PrintScreen button available as a hotkey?
When i want to make it a HotKey nothing happens,

Is this is a bug, or not implented yet?

Thanks for the great work already!

@vicnevicne
Copy link
Contributor

Indeed the PrintScreen does not seem to be recognized... Weird.
The lib I'm using once had a bug about that key, but it was fixed... 8 years ago - tulskiy/jkeymaster#9 - so I guess it is a configuration mistake on my side...
I'll have a look.

@vicnevicne
Copy link
Contributor

I didn't see anything particular in the configuration that could explain why that key goes unnoticed (particularly if used with non-nativly captured modifiers, such as Ctrl+PrintScreen, so I opened an issue with JKeyMaster. We'll see how it turns out.
By the way, can you tell me more about your setup ? Do you use a full 105-key keyboard or a laptop one where PrintScreen requires a FN key to be pressed ?
Also, can you tell me what OS version you're using ?
Thanks.

@elbertb
Copy link
Author

elbertb commented Jul 29, 2020

thanks for the quick investigation.

Yes, I am using a full 105-Keyboard and I have Windows 10 Pro 64Bit.
If u need any more information let me know.

@vicnevicne
Copy link
Contributor

vicnevicne commented Jul 30, 2020

I have a good news and a bad news.

The good news is that the key is handled correctly by the JKeyMaster lib. It's just not configurable from a text field as I did.
So there is a simple workaround:
Exit Ginj, edit the configuration file at :
%USERPROFILE%\.Ginj\settings.properties
and change the capture.hotkey line to read as follows:
capture.hotkey=PRINTSCREEN
Then save and restart Ginj. It should now have the PrintScreen key as capture hotkey.

Note that you can prefix it with "ctrl", "alt" or "shift" (or a combination thereof), all separated by blanks. E.g.
capture.hotkey=ctrl alt PRINTSCREEN
(mind the letter capitalization)

The bad news is that as I said, there is no easy way to capture it from a textfield, so the only solution I see now is to switch to a different UI (with checkboxes for modifiers + a drop down for keys), but I find it ugly, so I think I'll leave it as is for now...

@elbertb
Copy link
Author

elbertb commented Jul 30, 2020

Hi,

Thanks for searching for a solution, I changed it in the config file and now it works for me.

For me it is Ok now, so i want to thank you!
If i have more bugs, or improvements i will let you know.

@mae-byte-u
Copy link

The good news is that the key is handled correctly by the JKeyMaster lib. It's just not configurable from a text field as I did. So there is a simple workaround: Exit Ginj, edit the configuration file at : %USERPROFILE%\.Ginj\settings.properties and change the capture.hotkey line to read as follows: capture.hotkey=PRINTSCREEN Then save and restart Ginj. It should now have the PrintScreen key as capture hotkey.

I'm having this issue as well and tried your solution. I opened settings.properties in Notepad (as I have no other app right now to open it) and cannot find the capture.hotkey line. I'm running Windows 10 Pro 64 as well.

@vicnevicne
Copy link
Contributor

(Notepad should open the properties file no problem)
Hmm, maybe you never set a hotkey ?
Could you try first setting a hotkey (say CTRL SHIFT P) using the options dialog ? Then click OK and Exit Ginj, and finally open the ginj.properties file and the line should appear.
Alternately, please just exit Ginj and add the line to the properties file.
Can you confirm it works ?
KR,
Vicne

@mae-byte-u
Copy link

Ayyy it works! Thank you.

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