D10407: Disable positioning in popup folderviews

2018-02-19 Thread Andras Mantia
amantia closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D10407 To: amantia, hein Cc: davidedmundson, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10407: Disable positioning in popup folderviews

2018-02-11 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/D10407 To: amantia, hein Cc: davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg,

D10407: Disable positioning in popup folderviews

2018-02-10 Thread Andras Mantia
amantia added a comment. Yes, I have commit access since some time. ;) REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D10407 To: amantia, hein Cc: davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol,

D10407: Disable positioning in popup folderviews

2018-02-09 Thread David Edmundson
davidedmundson added a comment. Looks sensible. Do you have commit access? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D10407 To: amantia, hein Cc: davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas,

D10407: Disable positioning in popup folderviews

2018-02-09 Thread Andras Mantia
amantia added a reviewer: hein. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D10407 To: amantia, hein Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10407: Disable positioning in popup folderviews

2018-02-09 Thread Andras Mantia
amantia created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. amantia requested review of this revision. REVISION SUMMARY We need to use the overriden sortMode, not the model's sort mode. The == -> === changes are just