Review Request: "add virtual desktop" action for pager

[Aaron Seigo]

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

Ship it!


you know what would be cool as a "next step" after this? to have "remove this virtual desktop" when right clicking on a desktop in the pager ... add, remove, easy peasy? :)

- Aaron


On 2009-10-17 02:08:45, Chani wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1872/
> -----------------------------------------------------------
> 
> (Updated 2009-10-17 02:08:45)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> just a simple addition: you can now increment the number of virtual desktops from the pager's contextmenu. nice and convenient. :)
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/desktop/applets/pager/pager.h 1035858 
>   /trunk/KDE/kdebase/workspace/plasma/desktop/applets/pager/pager.cpp 1035858 
> 
> Diff: http://reviewboard.kde.org/r/1872/diff
> 
> 
> Testing
> -------
> 
> works fine for me, even with >1 pager running, although I don't fully understand the code I pasted.
> 
> 
> Thanks,
> 
> Chani
> 
>