D24839: Only show "Unlock Widgets" in the UI, not "Lock Widgets"

2019-10-29 Thread Björn Feber
This revision was automatically updated to reflect the committed changes. Closed by commit R120:f9a664cfd898: Only show "Unlock Widgets" in the UI, not "Lock Widgets" (authored by GB_2). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24839?vs=68510&id

D24839: Only show "Unlock Widgets" in the UI, not "Lock Widgets"

2019-10-26 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. This will unclutter the context menu a bit, +1 REPOSITORY R120 Plasma Workspace BRANCH arcpatch-D24839 REVISION DETAIL https://phabricator.kde.org/D24839 To: GB_2, #plasma, #vdg, ngraham, mart Cc: broulik, mart, ngraham, #vdg, p

D24839: Only show "Unlock Widgets" in the UI, not "Lock Widgets"

2019-10-26 Thread Björn Feber
GB_2 added a comment. Ping. REPOSITORY R120 Plasma Workspace BRANCH arcpatch-D24839 REVISION DETAIL https://phabricator.kde.org/D24839 To: GB_2, #plasma, #vdg, ngraham, mart Cc: broulik, mart, ngraham, #vdg, plasma-devel, #plasma, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas,

D24839: Only show "Unlock Widgets" in the UI, not "Lock Widgets"

2019-10-22 Thread Björn Feber
GB_2 updated this revision to Diff 68510. GB_2 added a comment. Avoid negating entire boolean statements REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24839?vs=68480&id=68510 BRANCH arcpatch-D24839 REVISION DETAIL https://phabricator.kde.org

D24839: Only show "Unlock Widgets" in the UI, not "Lock Widgets"

2019-10-22 Thread Kai Uwe Broulik
broulik added inline comments. INLINE COMMENTS > menu.cpp:146 > +// Only show "Unlock Widgets" in the UI, not "Lock Widgets" > +if (!(name == QLatin1String("lock widgets") && > c->corona()->immutability() == Plasma::Types::Mutable)) { > +action

D24839: Only show "Unlock Widgets" in the UI, not "Lock Widgets"

2019-10-21 Thread Björn Feber
GB_2 updated this revision to Diff 68480. GB_2 added a comment. Rebase REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24839?vs=68468&id=68480 BRANCH arcpatch-D24839 REVISION DETAIL https://phabricator.kde.org/D24839 AFFECTED FILES containm

D24839: Only show "Unlock Widgets" in the UI, not "Lock Widgets"

2019-10-21 Thread Nathaniel Graham
ngraham added a comment. Also needs a rebase REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D24839 To: GB_2, #plasma, #vdg Cc: ngraham, #vdg, plasma-devel, #plasma, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himc

D24839: Only show "Unlock Widgets" in the UI, not "Lock Widgets"

2019-10-21 Thread Björn Feber
GB_2 created this revision. GB_2 added reviewers: Plasma, VDG. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. GB_2 requested review of this revision. REVISION SUMMARY Not needed for normal users with the new edit mode system. TEST PLAN Lock the widgets before compili