Review Request: Wrong new rows position on Kickoff (simpleapplet view)

Arthur Mello arthur.mello85 at gmail.com
Mon Mar 30 19:37:13 CEST 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/480/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

On kickoff (simpleapplet view) when a row is inserted on a used model its position will be calculated based on other entries from same model. If no such entry is found the new one will be appended on menu. As a new user, the RecentlyUsed model will not have any rows when it is added to kickoff, so any app used from there will be placed at the bottom of the menu.

This patch add a not visible QAction to work as a model header on menu. If no other entry from the same model is found we use
 this header to put the new action on the right place.


Diffs
-----

  /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/simpleapplet/menuview.cpp 946987 

Diff: http://reviewboard.kde.org/r/480/diff


Testing
-------


Thanks,

Arthur



More information about the Plasma-devel mailing list