Review Request: Fix "Add to desktop" from Kickoff when you have several virtual desktops and you enable "Different widgets for each desktop"

[Aaron J. Seigo]

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

Ship it!


tiny ws issue, but otherwise ok.

and why containment starts from 0 -> because that's where counting starts. that virtual desktops in KWindowSystem start at '1' is pretty silly, really. otherwise we end up with everything else being zeroth counted as per usual, except desktops which are counted from 1. i chose not to extend that oddity into plasma code.


plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp
<http://git.reviewboard.kde.org/r/103645/#comment8045>

    spaces around the '-' please :)


- Aaron J. Seigo


On Jan. 6, 2012, 9:11 p.m., Anne-Marie Mahfouf wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103645/
> -----------------------------------------------------------
> 
> (Updated Jan. 6, 2012, 9:11 p.m.)
> 
> 
> Review request for Plasma and Aaron J. Seigo.
> 
> 
> Description
> -------
> 
> From Kickoff using "Add to desktop" when you have several virtual desktops and you enable "Different widgets for each desktop" in the pager settings. KWindowSystem starts counting from 1 and Plasma from 0
> 
> Without this fix "Add to desktop" adds to the next desktop or does not add if you're on the last desktop. 
> 
> 
> This addresses bug https://bugs.kde.org/show_bug.cgi?id=290368.
>     http://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=290368
> 
> 
> Diffs
> -----
> 
>   plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp cf12903 
> 
> Diff: http://git.reviewboard.kde.org/r/103645/diff/diff
> 
> 
> Testing
> -------
> 
> Local tests as thorough as I could do.
> 
> 
> Thanks,
> 
> Anne-Marie Mahfouf
> 
>

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