Don't move back items to second screen if they were manually moved
ClosedPublic

Authored by amantia on Dec 13 2017, 2:03 PM.

Details

Summary

When a secondary screen is removed, the items from there are moved to
the first screen. If the screen is plugged back, the items are moved back.
This patch does not move back the items on screen reconnect if meantime they were manually
moved around on the first screen.

Diff Detail

Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
amantia requested review of this revision.Dec 13 2017, 2:03 PM
amantia created this revision.
amantia set the repository for this revision to R119 Plasma Desktop.
Restricted Application added a project: Plasma. · View Herald TranscriptDec 13 2017, 2:08 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
hein accepted this revision.Dec 13 2017, 2:18 PM
This revision is now accepted and ready to land.Dec 13 2017, 2:18 PM
amantia closed this revision.Dec 14 2017, 11:50 AM
mwolff added inline comments.Dec 19 2017, 1:28 PM
containments/desktop/plugins/folder/screenmapper.cpp
177

doesn't this work too:

for (auto &names : m_itemsOnDisabledScreensMap) {
    names.removeAll(name);
}