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

[Craig Duquette]

I moved the i18n translation code outside of the addToolBoxTool()
function. Jason suggested making the change and I agree with it.

On 12/28/07, Craig Duquette <cduquette at gmail.com> wrote:
> Alright I'm going to try and do things one at a time, wait for a
> critique, make changes and then upload the changes to SVN instead of
> having 1 patch that does a lot of stuff at once :D
>
> This patch I'm attaching changes the toolbox buttons to icons. Text is
> stored underneath for the time being until more work is done on
> Plasma::Icon allowing me to put text on the side.
>
> Once I get approval on this patch I'll work on fixed-positioning on
> the toolbox and positioning of the widgets.
>
> ~Craig AKA gamma
>
> On 12/27/07, Chani <chanika at gmail.com> wrote:
> >
> > > i like the direction of this. it will need text ... but i do like the
> > icons
> > > ending up on the arc.. maybe the text can appear outside the arc and the
> > > icons in the arc.
> >
> > gamma was talking about two possibilities in irc, iirc: putting text
> beside
> > the icons (should be supported by icons real soon) or having one central
> > textlabel that shows the text of the icon under the mouse. tooltips turned
> > out to be too ugly in this case.
> > oh, and for text beside icons... he'd need it to show up on the left, not
> > the
> > right, mart.
> >
> > --
> > This message brought to you by evyl bananas, and the number 3.
> > www.chani3.com
> >
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: icons-122707.patch
Type: text/x-patch
Size: 3330 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20071228/c1601b03/attachment.bin