Differential D11168

[AppMenu Applet] Support actions without menu
ClosedPublic
Actions

Authored by broulik on Mar 9 2018, 8:07 AM.

Details

Reviewers
davidedmundson
Group Reviewers
Plasma
Commits
R120:640042ce0771: [AppMenu Applet] Support actions without menu
Summary

Applications could place actions in the main menu bar that do not have a menu.
Also restore the checked binding when clicked to avoid it highlighting even though there is no menu. Also might fix some spurious highlighting that I've seen when switching between windows.

Test Plan

Had an application with actions without menu and could trigger them just fine.

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.Mar 9 2018, 8:07 AM
Restricted Application added a project: Plasma. · View Herald TranscriptMar 9 2018, 8:07 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Mar 9 2018, 8:07 AM
davidedmundson accepted this revision.Mar 9 2018, 1:14 PM
This revision is now accepted and ready to land.Mar 9 2018, 1:14 PM
Closed by commit R120:640042ce0771: [AppMenu Applet] Support actions without menu (authored by broulik). · Explain WhyMar 9 2018, 1:21 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 29081

View Options

applets/appmenu/lib/appmenuapplet.cpp

Loading...
View Options

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

Loading...