<div>broulik created this revision.<br />
broulik added a reviewer: Plasma.<br />
broulik set the repository for this revision to rPLASMAINTEGRATION Integration for Qt applications in Plasma.<br />
Restricted Application added a project: Plasma.<br />
Restricted Application added a subscriber: plasma-devel.</div><br /><div><strong>REVISION SUMMARY</strong><div><p>While KXmlGui makes sure to prevent the user from hiding toolbars when not allowed,<br />
when right-clicking an empty space in a QMainWindow, such as the menu bar, will yield<br />
a menu created by Qt, which knows nothing about Kiosk restrictions, listing all toolbar<br />
toggle actions of this window.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>I can no longer hide the main toolbar in Dolphin or Gwenview.</p>

<p>See comment in the code for an explanation what it actually does :)<br />
In principle we could also think of enforcing other KAuthorized stuff here but both KXmlGui and KActionCollection take care of most of it.</p>

<p>How can I simulate the KAuthorized stuff for an autotest? Is there a KConfig kdeglobals test mode or something like that?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMAINTEGRATION Integration for Qt applications in Plasma</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1644" rel="noreferrer">https://phabricator.kde.org/D1644</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/CMakeLists.txt<br />
src/platformtheme/CMakeLists.txt<br />
src/platformtheme/kdeplatformtheme.cpp<br />
src/platformtheme/kdeplatformtheme.h<br />
src/platformtheme/kioskintegration.cpp<br />
src/platformtheme/kioskintegration.h</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>plasma-devel, sebas<br /></div>