Differential D17107

global menu,rename menuHidden to visible
ClosedPublic
Actions

Authored by mvourlakos on Nov 22 2018, 6:53 PM.

Details

Reviewers
mart
davidedmundson
Group Reviewers
Plasma
Commits
R120:5779dde86fe7: global menu,rename menuHidden to visible
Summary

--as @mart requested we can rename "menuHidden" property
to "visible" in order to follow Qt naming

Test Plan

--check that global menu applet is functioning the same
way as when using removed menuHidden property

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.
mvourlakos created this revision.Nov 22 2018, 6:53 PM
Restricted Application added a project: Plasma. · View Herald TranscriptNov 22 2018, 6:53 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mvourlakos requested review of this revision.Nov 22 2018, 6:53 PM
mart accepted this revision.Nov 23 2018, 11:00 AM
This revision is now accepted and ready to land.Nov 23 2018, 11:00 AM
Closed by commit R120:5779dde86fe7: global menu,rename menuHidden to visible (authored by mvourlakos). · Explain WhyNov 23 2018, 12:22 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 46074

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...