Plasma theming: Re-using existing widget SVGs

2010-07-13 Thread Ingomar Wesp
Hi, just a quick question regarding theming: The quicklaunch applet currently uses a KIcon for displaying the arrow that triggers/hides its popup, which obviously isn't theme aware and might look bad in some themes due to contrast issues. Since the graphics for a nice extender arrow are alread

Re: Plasma theming: Re-using existing widget SVGs

2010-07-13 Thread Marco Martin
On Tue, Jul 13, 2010 at 3:58 PM, Ingomar Wesp wrote: > Hi, > > just a quick question regarding theming: > > The quicklaunch applet currently uses a KIcon for displaying the arrow that > triggers/hides its popup, which obviously isn't theme aware and might look bad > in some themes due to contrast

Re: Plasma theming: Re-using existing widget SVGs

2010-07-13 Thread Ingomar Wesp
Marco Martin wrote: > i'm not sure the systray one would be the best choice as opposed to > the arrows svg [...] Ah, completely overlooked that one. I guess going for the more generic arrows is the better choice then. Unfortunately, this also adds a bit of visual inconsistency for users that h

Re: Plasma theming: Re-using existing widget SVGs

2010-07-13 Thread Aaron J. Seigo
On July 13, 2010, Ingomar Wesp wrote: > Marco Martin wrote: > > i'm not sure the systray one would be the best choice as opposed to > > the arrows svg [...] > > Ah, completely overlooked that one. > > I guess going for the more generic arrows is the better choice then. > Unfortunately, this also

Re: Plasma theming: Re-using existing widget SVGs

2010-07-13 Thread Marco Martin
On Tuesday 13 July 2010, Aaron J. Seigo wrote: > On July 13, 2010, Ingomar Wesp wrote: > > Marco Martin wrote: > > > i'm not sure the systray one would be the best choice as opposed to > > > the arrows svg [...] > > > > Ah, completely overlooked that one. > > > > I guess going for the more generi

Re: Plasma theming: Re-using existing widget SVGs

2010-07-15 Thread Ingomar Wesp
Marco Martin wrote: > [...] we could do the following: > > use arrows.svg for the systray (so old themes wouldn't be completely > broken, they would have the arrows) > then provide smaller versions with like a 16-16-left element and so forth, > so the arrows would look nice also with sizes as lit

Re: Plasma theming: Re-using existing widget SVGs

2010-07-15 Thread Marco Martin
On Thursday 15 July 2010, Ingomar Wesp wrote: > Marco Martin wrote: > > [...] we could do the following: > > > > use arrows.svg for the systray (so old themes wouldn't be completely > > broken, they would have the arrows) > > then provide smaller versions with like a 16-16-left element and so > >

Re: Plasma theming: Re-using existing widget SVGs

2010-07-15 Thread Ingomar Wesp
Marco Martin wrote: > the theme itself will be able to pick the proper ones optimized for a > certain resolution if present, not something one has to worry about :) Nifty. I knew something like this worked for KIcon, but it never occurred to me that Plasma::Theme / Plasma::SVG could do that as we