search

Skycoder42 / QHotkey

A global shortcut/hotkey for Desktop Qt-Applications
BSD 3-Clause "New" or "Revised" License
552 stars 162 forks source link

Add the ability to check if the platform is supported #27

Closed Shatur closed 4 years ago

Shatur commented 4 years ago

Thanks for your awesome library! I have a suggestion. Since hotkeys for Linux work only for X11, then could you add a function that checks if the platform is supported? This can be convenient if I want to show the user in the UI that hotkeys cannot be registered. This can be implemented by adding a function that returns true for Windows and Mac, and QX11Info::isPlatformX11() on Linux. I can make a PR if you approve my suggestion.

Skycoder42 commented 4 years ago

Sounds good to me. PRs are always welcome, so please, go ahead and create one!