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

Review request for Plasma and Lukas Appelhans.


Summary
-------

This simple patch simply adds the chance to show also the app name under the 
icon. The only thing i have a doubt about is that after choosing to show also 
the application name the icon size option in the configuration dialog changes 
its meaning becoming: "icon widget size".

One possible solution to this might be doing something like this: 
m_preferredIconSize = 
relevant_iconWidget->sizeFromIconSize(m_preferredIconSize).width(); when icon 
names are to be shown.

Waiting for suggestions :)


Diffs
-----

  trunk/KDE/kdebase/workspace/plasma/applets/quicklaunch/quicklaunchApplet.h 
1016222 
  trunk/KDE/kdebase/workspace/plasma/applets/quicklaunch/quicklaunchApplet.cpp 
1016222 
  trunk/KDE/kdebase/workspace/plasma/applets/quicklaunch/quicklaunchConfig.ui 
1016222 

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


Testing
-------


Thanks,

Alessandro

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to