----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/256/#review448 -----------------------------------------------------------
Ship it! - Aaron On 2009-03-08 15:22:38, Christian Mollekopf wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/256/ > ----------------------------------------------------------- > > (Updated 2009-03-08 15:22:38) > > > Review request for Plasma, Aaron Seigo and Marco Martin. > > > Summary > ------- > > This patch replaces the currently used TasksMenu with a Plasma::Dialog which > shows the same LayoutWidget which is also used in the expanded group. > This makes it possible to use the right click menu on items in the popup. > I couldn't find any drawbacks compared to the previous TasksMenu. > Drag&Drop support for sorting and grouping is not implemented though also not > really neccessary (but i think it would be possible). > > > Diffs > ----- > > /trunk/KDE/kdebase/workspace/plasma/applets/tasks/layoutwidget.cpp 936929 > /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.h 936929 > /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.cpp 936929 > > Diff: http://reviewboard.kde.org/r/256/diff > > > Testing > ------- > > Tested locally. > unfortunately i don't get theming with my svn installation so i couldn't test > if it looks nice. > > > Thanks, > > Christian > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
