-
Notifications
You must be signed in to change notification settings - Fork 112
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
Add a shortcut editor #1325
Comments
Do you think we should turn Strings should be friendly to QKeySequence. The only Qt signal provided is Should be sane don't you think? |
No extra shared library is needed for such jobs. |
In no way did I mention the term "extra". Personally I see an API more suitable than message bus for this task. How do you plan to record a shortcut? |
No need to a plan; Qt does it. See lximage-qt's code, for example. |
Okay 👍 |
The shortcut editor could be inside the
With that entry I could set the shortcut |
Self-explanatory but not straightforward because several shortcuts are defined inside the code and should be excluded from the future shortcut editor (I've encountered and handled the same situation elsewhere).
The text was updated successfully, but these errors were encountered: