Review Request: Make the task manager items move in realtime when user manually arranges items

Aaron J. Seigo aseigo at kde.org
Tue Oct 25 13:18:12 UTC 2011



> On Oct. 25, 2011, 12:48 p.m., Aaron J. Seigo wrote:
> > plasma/desktop/applets/tasks/taskgroupitem.cpp, lines 1037-1038
> > <http://git.reviewboard.kde.org/r/102833/diff/1/?file=38602#file38602line1037>
> >
> >     i wonder what happens here when you drag an entry from one tasks widget to another? :)
> 
> Shantanu Tushar Jha wrote:
>     Well in that case the corresponding entry moves even in the other tasks widget, which is the same behavior as earlier. I'm not sure if thats a problem, is it?

probably, as that's how it was before. just something i noticed while reviewing. shouldn't hold this patch up at all, though.


- Aaron J.


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102833/#review7601
-----------------------------------------------------------


On Oct. 11, 2011, 6:19 p.m., Shantanu Tushar Jha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102833/
> -----------------------------------------------------------
> 
> (Updated Oct. 11, 2011, 6:19 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
> Right now, if the user sets the Task Manager sorting to "manually", its possible to arrange the task items manually. However, the items move only after the mouse button is released.
> This patch makes the items move dynamically and follow the mouse, this helps to produce a more natural effect. Check the attached video for a demo.
> 
> 
> Diffs
> -----
> 
>   plasma/desktop/applets/tasks/taskgroupitem.h 7ca95ef 
>   plasma/desktop/applets/tasks/taskgroupitem.cpp 14053d6 
> 
> Diff: http://git.reviewboard.kde.org/r/102833/diff/diff
> 
> 
> Testing
> -------
> 
> Works perfectly in both horizontal and vertical configurations, and multiple rows. I didn't find any side-effects.
> 
> 
> Thanks,
> 
> Shantanu Tushar Jha
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20111025/bb01bb9c/attachment.html>


More information about the Plasma-devel mailing list