broulik created this revision. broulik added reviewers: Plasma, mart. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY Instead of duplicating the name and icon work, just bind it to the QAction itself. Also respect visibility of actions for KIOSK restriction. TEST PLAN It now uses System Settings app icon for "About System Settings" instead of "help-about", though. Placed the following in kdeglobals [KDE Action Restrictions] action/configure=false Action would no longer show up. It would never trigger as the action was disabled internally so this is purely cosmetic. REPOSITORY R124 System Settings REVISION DETAIL https://phabricator.kde.org/D8358 AFFECTED FILES sidebar/SidebarMode.cpp sidebar/SidebarMode.h sidebar/package/contents/ui/ActionMenu.qml sidebar/package/contents/ui/CategoriesPage.qml To: broulik, #plasma, mart Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart