D8515: Don't loop twice here

2017-10-30 Thread Laurent Montel 
This revision was automatically updated to reflect the committed changes.
Closed by commit R119:9a5e9be03fe6: Dont loop twice here (authored by 
mlaurent).

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8515?vs=21422=21552

REVISION DETAIL
  https://phabricator.kde.org/D8515

AFFECTED FILES
  containments/desktop/plugins/folder/foldermodel.cpp
  containments/desktop/plugins/folder/positioner.cpp

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

D8515: Don't loop twice here

2017-10-30 Thread Laurent Montel 
mlaurent added a comment.


  This one too ?:)

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D8515

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

D8515: Don't loop twice here

2017-10-27 Thread Laurent Montel 
mlaurent updated this revision to Diff 21422.
mlaurent added a comment.


  - Avoid twice loop here too

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8515?vs=21418=21422

BRANCH
  dont_loop_twice

REVISION DETAIL
  https://phabricator.kde.org/D8515

AFFECTED FILES
  containments/desktop/plugins/folder/foldermodel.cpp
  containments/desktop/plugins/folder/positioner.cpp

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

D8515: Don't loop twice here

2017-10-27 Thread Laurent Montel 
mlaurent added a reviewer: hein.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D8515

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