[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: [Panel-devel] [PATCH] Smaller toolbox with icons
From: Jason Stubbs <jasonbstubbs () gmail ! com>
Date: 2007-12-28 8:32:23
Message-ID: 200712281732.23245.jasonbstubbs () gmail ! com
[Download RAW message or body]
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
_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic