[shell] Add a dbus call to activate the "main" application launcher
ClosedPublic

Authored by graesslin on Aug 5 2016, 9:18 AM.

Details

Summary

ShellCorona provides a new DBus method "activateLauncherMenu". If that
is invoked it will look through all panels for an applet which has
X-Plasma-Provides with org.kde.plasma.launchermenu. If that also has
a global shortuct it gets activated and no further applet is searched.

The idea behind this DBus call is to provide a way to KWin to activate
the main appliction launcher when the Meta key is pressed.

Diff Detail

Repository
R120 Plasma Workspace
Branch
shell-dbus-launchermenu
Lint
No Linters Available
Unit
No Unit Test Coverage
graesslin updated this revision to Diff 5687.Aug 5 2016, 9:18 AM
graesslin retitled this revision from to [shell] Add a dbus call to activate the "main" application launcher.
graesslin updated this object.
graesslin edited the test plan for this revision. (Show Details)
graesslin added reviewers: Plasma, hein, mart.
Restricted Application added a project: Plasma. · View Herald TranscriptAug 5 2016, 9:18 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik added a subscriber: broulik.Aug 5 2016, 9:51 AM
broulik added inline comments.
shell/shellcorona.cpp
1635

++it?

1638

X-Plasma-Provides is a QStringList

graesslin marked 2 inline comments as done.Aug 5 2016, 12:56 PM
graesslin updated this revision to Diff 5694.Aug 5 2016, 12:57 PM

Addressed Kai's review comments

mart accepted this revision.Aug 8 2016, 3:01 PM
mart edited edge metadata.
mart added inline comments.
shell/shellcorona.cpp
1640

the way i can see it breaking is if an user has 2 menus with 2 different shortcuts (extremely unlikely and weird, but i don't get surprised by anything anymore)
it's fine for me, just to know the case exists

This revision is now accepted and ready to land.Aug 8 2016, 3:01 PM
This revision was automatically updated to reflect the committed changes.