Review Request: Add submenu support to QML shutdown dialog.

Lamarque Vieira Souza lamarque at
Sat Jul 14 06:41:57 BST 2012

This is an automatically generated e-mail. To reply, visit:

(Updated July 14, 2012, 5:41 a.m.)

Review request for KDE Runtime and Konstantinos Smanis.


. align menu entries to the left instead of center.
. add hover effect.
. add icon to the menu entry to indicate that it contains a submenu.
. close all submenus when one entry is clicked.
. add screenshot


Add support to show submenus in the new QML shutdown dialog. I think this patch can be improved, the GUI too. When I have more time I will go back to improve it, until there you can send suggestions.

The patch assumes rebootOptions now contains strings like:

submenu1 > subentry 1.1
submenu1 > subentry 1.2
submenu2 > subentry 2.1
submenu2 > subsubmenu 1 > subsubentry 2.1.1

The character '>' is the separator for submenus.

This addresses bug 297209.

Diffs (updated)

  ksmserver/themes/default/ContextMenu.qml 6f2f1fd 
  ksmserver/themes/default/MenuItem.qml 74bb03f 
  ksmserver/themes/default/main.qml 7e78761 



Submenus are created and it emits rebootRequested2 signal with the correct index.

Screenshots (updated)


Lamarque Vieira Souza

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the kde-core-devel mailing list