D6455: position the shellsurface immediately

2018-08-28 Thread David Edmundson
davidedmundson requested changes to this revision. davidedmundson added a comment. This revision now requires changes to proceed. This seems outdated. It also doesn't seem to be an issue? Possibly fixed by us having the extra roundtrip with the XDG shell configure REPOSITORY R120 Plasma

D6455: position the shellsurface immediately

2017-07-11 Thread Martin Flöser
graesslin added a comment. What I don't like about it is that KWin will get and react on two events with one having invalid coordinates. We get an event for the change of x and at that point y might be invalid. KWin will position the window there and that can result in mess (e.g. all

D6455: position the shellsurface immediately

2017-07-11 Thread Marco Martin
mart added a reviewer: graesslin. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D6455 To: mart, #plasma, graesslin Cc: davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

D6455: position the shellsurface immediately

2017-06-30 Thread David Edmundson
davidedmundson added a comment. > otherwise it's too late too late for what? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D6455 To: mart, #plasma Cc: davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts,

D6455: position the shellsurface immediately

2017-06-30 Thread Marco Martin
mart created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY when the window gets moved, position the shellsurface immediately rather than in an event: otherwise it's too late and having a moment in