[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: [Panel-devel] trashcan applet
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2007-11-29 14:45:55
Message-ID: 200711290745.55803.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Thursday 29 November 2007, Marco Martin wrote:
> On Nov 29, 2007 2:08 AM, Aaron J. Seigo <aseigo@kde.org> wrote:
> > it's a bug. trying to track it down.. Plasma::Icon is pretty old code at
> > this
> > point and it's rather broken =/
>
> it seems to be fixed if i explicitly put the icon into the layout like in
> this patch, that i think it makes sense, even if i always have the feeling
> that sizeHint of Icon doesn't work right

hm. interesting. that's a good clue.

> > it already has no background, text under it would be awesome. actually,
> > the
> > text under it needs to go into Plasma::Icon itself, really.
>
> uhmmm, in the trash i use the setText method of the icon, that maybe should
> be refinied a little bit (often impossible to read, maybe using a
> drop-shadow?) but decining what the text is i think should be decided from
> the applet, for example for the trash is trash and the number of items, for
> the launcher i would say the filename part of the url or if it points to a
> desktop file the name stored in it

ah yes, sorry, i misunderstood what you were saying; yes, you are absolutely 
correct here.

> maybe in the next few days i will see if i can understand something of the
> icon class and if i am able to iron out some issues

awesome =)


-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech

[Attachment #5 (application/pgp-signature)]

_______________________________________________
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]