[Differential] [Changed Subscribers] D2356: [shell] Add a dbus call to activate the "main" application launcher

[broulik (Kai Uwe Broulik)]

broulik added inline comments.

INLINE COMMENTS

> shellcorona.cpp:1635
> +{
> +    for (auto it = m_panelViews.constBegin(), end = m_panelViews.constEnd(); it != end; it++) {
> +        const auto applets = it.key()->applets();

++it?

> shellcorona.cpp:1638
> +        for (auto applet : applets) {
> +            if (applet->pluginInfo().property("X-Plasma-Provides").toString() == QStringLiteral("org.kde.plasma.launchermenu")) {
> +                if (!applet->globalShortcut().isEmpty()) {

X-Plasma-Provides is a QStringList

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D2356

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #plasma, hein, mart
Cc: broulik, plasma-devel, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160805/de1c98c7/attachment.html>