- Don't autostart KRunner, use actions in the desktop file to
define global shortcuts and actions
- install its desktop file in share/kglobalaccel to make shortcuts default
- expand commandline options to permit more control via commandline
davidedmundson | |
mart |
Plasma |
define global shortcuts and actions
works, as expected except
No Linters Available |
No Unit Test Coverage |
Needs the things you pointed out in the comment, but otherwise looks good.
Can you confirm you can set shortcuts in systemsettings correctly?
Overall fine with me.
What hapens to places like Plasma's context menu that show "Run command (Alt+F2)"? What shortcut will they show and do they still work? Same for typing on the desktop.
krunner/main.cpp | ||
---|---|---|
71 | Use initializer list {QStringLiteral("c"), QStringLiteral("clipboard")} | |
77 | Why no i18n() here? Also, QStringLiteral("query") | |
108 | This stuffin the lambda is somewhat duplicated, isn't it? |
they still work, tough typing on desktop may lose some letters.. i wonder if would make sense to catch them plasmashell-side then launch krunner with the query as parameter...
delaying the first show seems to fix the focus issue.
a fixed timer of 100ms is of course not a solution but may give some insight in what the problem actually is (a 0 delay timer doesn't work either)
krunner/update/krunnerglobalshortcuts.cpp | ||
---|---|---|
34 |
File=kglobalshortcutsrc in the .upd. The syntax allows for quite a lot of manipulation.
kconf_update runs, the config updates, kglobalacceld is still running so later syncs undoing your script. Sorry :/ |
I have a plan for a kconf_update v2 which would resolve the problem above nicely
Probably means this would be 5.14.
Otherwise you can copy my 4ae36ddddaee91a23dcb0736418295269da14152 in powerdevil