[Panel-devel] [PATCH] Smaller toolbox with icons

Jason Stubbs jasonbstubbs at gmail.com
Fri Dec 28 09:32:23 CET 2007


On Friday 28 December 2007 17:07:32 Craig Duquette wrote:
> Chani people actually read documentation? :P I added that line back
> in. I switched the QObject to a Plasma::Icon.

One tiny nit-picky thing so no need to repost a patch, but:

+    tool->setMinimumSize(iconSize);
+    tool->setMaximumSize(iconSize);
+    tool->resize(tool->sizeHint());

Is there any reason to not use iconSize rather than tool->sizeHint() ? I'm 
guessing that it returns the same value anyway, but looks out of place and 
begs the eternal question "why is it so?" ;)

-- 
Jason Stubbs


More information about the Panel-devel mailing list