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

List:       kde-panel-devel
Subject:    Re: Patch for system tray layouting
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2008-06-15 19:32:19
Message-ID: 200806151332.19931.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Sunday 15 June 2008, Marco Martin wrote:
> On Saturday 14 June 2008, Bernhard wrote:
> > Hi!
> >
> > I can't say anything about your patch in generally because I'm just not
> > familiar with the internals of KDE yet ... but (and I don't know if it's
> > your fault) why the hell is the size of the systray icons hardcoded??

short answer: the fd.o sys tray spec is really bad. it was ok for the time it 
was invented (kde1), but we've really outgrown it.

> > Shouldn't there be a global setting for the icon size?

in KDE there is. or at least, the icon size is set by the KSystemTrayIcon 
class.

> the problem is, among the nastiness of the systray that the mini windows
> that rapresent systray icons are 22x22, no matter what and we can't do
> nothing about that

actually, the problem is even worse than that =/

in KDE, our icons are 22x22. but the systray spec itself says nothing about 
that. in fact, i know of at least one gtk+ app that puts a widget into the 
systray that contains a horizontal layout that contains an icon and a text 
label! this thing can easily be 50-100px wide.

this is why i eventually re-wrote kicker's sytray layouting to allow items to 
span multiple rows and take that into consideration when doing layouts.

the systray spec sucks so badly, in so many ways. it makes me cry every time i 
look at it. =)

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


["signature.asc" (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] 

Configure | About | News | Add a list | Sponsored by KoreLogic