Review Request 125694: Use the correct old screen to shift panels to new screen

David Kahles david.kahles96 at gmail.com
Mon Oct 19 11:10:30 UTC 2015


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

(Updated Oct. 19, 2015, 11:10 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit 20778e83026d5390230bb3b2726f9c722878ba78 by David Kahles to branch master.


Repository: plasma-workspace


Description
-------

To move the panels of screen [i] to the next screen, we should query for
the panels of DesktopView [i], not of DesktopView [i + 1]. Otherwise the
panels aren't shifted to the new screen. Thus QT shifted the
panels itself, but hid them afterwards. Besides the panels were not
visible, it raised an assert in screenInvariants().


Diffs
-----

  shell/shellcorona.cpp 9185fb58a5e7ca09f916bbc5a231e7220a562c85 

Diff: https://git.reviewboard.kde.org/r/125694/diff/


Testing
-------

The panel now gets shifted correctly when unplugging a screen. I tested it only with 2 screens, i'm not sure whether this loop works with more than 2 screens, this needs more investigation..


Thanks,

David Kahles

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


More information about the Plasma-devel mailing list