Review Request 127046: Move popup menu image actions into a submenu

[Jonathan Marten]

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

Review request for KDE Frameworks and KDE Usability.


Repository: khtml


Description
-------

The popup menu over an image (with actions "Save Image As..." etcetera, see screen shot) has in total 6 image actions which are not frequently used but make the menu very long.  Moving these actions onto a submenu makes the top level menu smaller and more logically grouped.


Diffs
-----

  src/khtml_ext.cpp 0f522f4 

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


Testing
-------

Built KHTML with these changes, tested in Konqueror.


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

Popup menu before
  https://git.reviewboard.kde.org/media/uploaded/files/2016/02/11/55c12311-73a3-472a-8a9a-6b6b9fce9de7__before.png
Popup menu after
  https://git.reviewboard.kde.org/media/uploaded/files/2016/02/11/313dbe3b-efb4-475c-bf76-8516f5bc85b4__after.png


Thanks,

Jonathan Marten

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160211/627a790b/attachment.html>