Differential D6336

[AppMenu Applet] Wire up DBusMenuImporter::actionActivationRequested
ClosedPublic
Actions

Authored by broulik on Jun 22 2017, 5:38 AM.

Details

Reviewers
davidedmundson
Group Reviewers
Plasma
Commits
R120:92a7089e9a10: [AppMenu Applet] Wire up DBusMenuImporter::actionActivationRequested
Summary

Ensures that pressing e.g. Alt+F will popup the "File" menu.

BUG: 376726
FIXED-IN: 5.10.3

Test Plan

I upgraded to Qt 5.9 and it was working just fine with title bar menu button but didn't with the app menu applet. Now it does.

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Jun 22 2017, 5:38 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJun 22 2017, 5:39 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson accepted this revision.Jun 22 2017, 7:40 AM
This revision is now accepted and ready to land.Jun 22 2017, 7:40 AM
Closed by commit R120:92a7089e9a10: [AppMenu Applet] Wire up DBusMenuImporter::actionActivationRequested (authored by broulik). · Explain WhyJun 22 2017, 8:40 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 15730

View Options

applets/appmenu/package/contents/ui/main.qml

Loading...
View Options

applets/appmenu/plugin/appmenumodel.h

Loading...
View Options

applets/appmenu/plugin/appmenumodel.cpp

Loading...