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);
  }

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-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, 
abetts, sebas, apol, mart

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
  https://phabricator.kde.org/D9313

To: amantia, #plasma, hein, mwolff
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart