simplelauncher review request

[David Johnson]

The simplelauncher classic menu looks fine under the Oxygen style, but under 
most other styles the margins around the menu are too small. While this is 
okay for application menus, it looks odd for an application launcher.

Under the Oxygen style there is ample space around the launcher menu, but 
under Plastique, Cleanlooks, Phase, etc., there is no space, making the menu 
look cramped.

I am attaching a patch that checks the style's menu panel width, and if too 
small sets contents margin for the menu to make up for it. This change is 
transparent for the default Oxygen style, and only affects styles with a menu 
panel width of less than 4.

If this is suitable, I can check it in.

Thank you,

-- 
David Johnson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: menuview.cpp.diff
Type: text/x-diff
Size: 1109 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20090131/4ace294c/attachment.bin