Review Request 127971: [KToolBar] Disable toggleViewAction() when actions/options_show_toolbar is restricted

Kai Uwe Broulik kde at privat.broulik.de
Thu May 19 21:37:51 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127971/
-----------------------------------------------------------

(Updated Mai 19, 2016, 9:37 nachm.)


Review request for KDE Frameworks and Plasma.


Changes
-------

* Move KAuthorized check to the other one in Private::init


Repository: kxmlgui


Description
-------

While KXmlGui makes sure to prevent the user from hiding toolbars when not allowed, right-clicking an empty space in a QMainWindow, such as the menu bar, will yield a menu created by Qt, which knows nothing about Kiosk restrictions, listing all toolbar toggle actions of this window.


Diffs (updated)
-----

  autotests/ktoolbar_unittest.cpp d6c1e05 
  src/ktoolbar.cpp 8fcb9cb 

Diff: https://git.reviewboard.kde.org/r/127971/diff/


Testing
-------

Comes with a unittest.

I can no longer hide the main toolbar in Dolphin or Gwenview when this action is restricted by simply right-clicking the menu bar.


File Attachments
----------------

The Menu
  https://git.reviewboard.kde.org/media/uploaded/files/2016/05/19/076745b4-03a2-400f-9506-46999d4ce1e1__toolbartoggle.png


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160519/4f0cd316/attachment-0001.html>


More information about the Plasma-devel mailing list