Re: Review Request: Tasks applet: group popup with real task items

2009-03-10 Thread Aaron Seigo
--- 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: >

Re: Review Request: Tasks applet: group popup with real task items

2009-03-08 Thread Christian Mollekopf
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/256/ --- (Updated 2009-03-08 15:22:38.552515) Review request for Plasma, Aaron Seigo and M

Re: Review Request: Tasks applet: group popup with real task items

2009-03-08 Thread Christian Mollekopf
> On 2009-03-08 09:19:15, Marco Martin wrote: > > /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.cpp, line > > 565 > > > > > > using an event filter just to hide the window on lost focus seems a bit > > ov

Re: Review Request: Tasks applet: group popup with real task items

2009-03-08 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/256/#review406 --- Ship it! other a single issue it looks good (and an improvement for sur

Review Request: Tasks applet: group popup with real task items

2009-03-08 Thread Christian Mollekopf
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/256/ --- Review request for Plasma, Aaron Seigo and Marco Martin. Summary --- This pa