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@kde.org > https://mail.kde.org/mailman/listinfo/plasma-devel _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel