This fixes a wishlist issue in the KDE bug tracker : https://bugs.kde.org/show_bug.cgi?id=368765
BUG: 368765
Details
Details
Diff Detail
Diff Detail
- Repository
- R368 RSIBreak
- Branch
- tooltip (branched from master)
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 14859 Build 14877: arc lint + arc unit
Comment Actions
Having the possibility of setting a global shortcut is cool, but having one by default i think it may be a bit too much.
How would you feel about just making it be a global shortcut and letting people configure it from the config screen?
Comment Actions
That sounds reasonable. It should just be a matter of calling setDefaultShortcut instead, I think. Let me play around with it and update the diff.
Comment Actions
Instead of pre-assigning a global shortcut, this makes it just show up in the global shortcuts entry and lets the user enable and customize the key combination. I verified that enabling the shortcut works as expected.
Comment Actions
The "problem" with this is that you need to go to systemsettings and configure the shortcut from there.
Do you think you could look at spectacle's code and see how they do it so that they can show the configure global shortcuts from the app too?
Comment Actions
I see, you are suggesting a method to configure the shortcut within the app itself. I think I found an example of this in kompare - https://github.com/annulen/kompare/blob/master/kompare_shell.cpp#L438 - that seems to be the way to make this happen.
Comment Actions
KShorcutsEditor is what you wants, but you need the special options to show the global actions, as mentioned, looked at the spectacle code