Review Request 118548: Port libtaskmanager away from QDesktopWidget

Eike Hein hein at kde.org
Thu Jun 5 16:55:39 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118548/#review59350
-----------------------------------------------------------

Ship it!


Looks good.

- Eike Hein


On June 5, 2014, 12:35 a.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118548/
> -----------------------------------------------------------
> 
> (Updated June 5, 2014, 12:35 a.m.)
> 
> 
> Review request for Plasma, Eike Hein and Luca Beltrame.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> plasmoid.screen doesn't map to QDesktopWidget indexes anymore, therefore we need to port it.
> 
> This patch uses the screen geometry to figure out what's the screen and then passes around the screen rect so that we can filter out the screens that aren't inside if the user asks for it.
> 
> 
> Diffs
> -----
> 
>   libtaskmanager/taskmanager.cpp 27eeed7 
>   libtaskmanager/taskmanager.h e6ca735 
>   libtaskmanager/task.h 13a5a9c 
>   libtaskmanager/task.cpp 50ea1a6 
>   libtaskmanager/launcheritem.cpp 649caca 
>   libtaskmanager/groupmanager.h aa71bac 
>   libtaskmanager/groupmanager.cpp 83b39ef 
> 
> Diff: https://git.reviewboard.kde.org/r/118548/diff/
> 
> 
> Testing
> -------
> 
> I have played with it and seems to work.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

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


More information about the Plasma-devel mailing list