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

2017-12-19 Thread Milian Wolff
mwolff added inline comments. INLINE COMMENTS > screenmapper.cpp:177 > +{ > +for (auto it = m_itemsOnDisabledScreensMap.begin(); > + it != m_itemsOnDisabledScreensMap.end(); ++it) { doesn't this work too: for (auto : m_itemsOnDisabledScreensMap) { names.removeAll(name);

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

2017-12-14 Thread Andras Mantia
amantia closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D9313 To: amantia, #plasma, hein, mwolff Cc: ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

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

2017-12-13 Thread Eike Hein
hein accepted this revision. This revision is now accepted and ready to land. REPOSITORY R119 Plasma Desktop BRANCH master REVISION DETAIL https://phabricator.kde.org/D9313 To: amantia, #plasma, hein, mwolff Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg,

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

2017-12-13 Thread Andras Mantia
amantia added reviewers: Plasma, hein, mwolff. amantia set the repository for this revision to R119 Plasma Desktop. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REPOSITORY R119 Plasma Desktop REVISION DETAIL