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
Branch
menuVisible
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 5263
Build 5281: arc lint + arc unit
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 46032

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