Review Request: style on the taskbar group menu

Marco Martin notmart at gmail.com
Mon Nov 3 23:33:58 CET 2008


On Monday 03 November 2008, Fredrik Höglund wrote:
> On Monday 03 November 2008 17:58, Marco Martin wrote:
> > -----------------------------------------------------------
> > This is an automatically generated e-mail. To reply, visit:
> > http://reviewboard.vidsolbach.de/r/254/
> > -----------------------------------------------------------
> >
> > Review request for Plasma.
> >
> >
> > Summary
> > -------
> >
> > this makes the taskbar group menu to draw mostly like an extender, to
> > implement a mockup of the new theme it's just a subclass of the menu with
> > custom painting, still think in the future should be  the real tasks
> > widgets within a view, but i kinda feel it's a 4.3 thing :)
>
> With the text color hardcoded to white, this code will only work with dark
> themes.
ha! i have just plain forgotten that :(

> I also don't think it's a good idea to abuse the tasks-hover background for
> the highlighted item, since this imposes a significant design limitation on
> the theme in that the taskbar button background has to be drawn in such a
> way that it also works as the highlighted menu background. I think the
> theme should provide a separate element for highlighted menu items.
the idea was to resemble the task entries because in the future that would 
probably become a graphicsview with the actual tasks inside it
just an idea maybe won't be done in that way, but the idea was to make it more 
look like the taskbar than a menu...
if in the end will be decided to leave it as a menu like that yeah, an element 
just for menus could make sense yeah.

Cheers,
Marco Martin

> Looking at other systems it seems that Vista uses the same theme element
> for selected icons, items in treeviews, and popup menus.
>
> Fredrik
>
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel




More information about the Plasma-devel mailing list