D8268: Fix placement of KSplash

2017-10-15 Thread Martin Flöser
This revision was automatically updated to reflect the committed changes. Closed by commit R108:afe0a5c04116: Fix placement of KSplash (authored by graesslin). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D8268?vs=20657=20796#toc REPOSITORY R108 KWin CHANGES SINCE LAST UPDATE

D8268: Fix placement of KSplash

2017-10-15 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R108 KWin BRANCH ksplash-fix-5.11 REVISION DETAIL https://phabricator.kde.org/D8268 To: graesslin, #kwin, #plasma, davidedmundson Cc: plasma-devel, kwin, bwowk, ZrenBot, progwolff,

D8268: Fix placement of KSplash

2017-10-15 Thread Martin Flöser
graesslin added a comment. Addressed the other problem in https://phabricator.kde.org/D8304. As that is only master I would retarget this change here for master as well instead of 5.11. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D8268 To: graesslin, #kwin,

D8268: Fix placement of KSplash

2017-10-15 Thread Martin Flöser
graesslin added a comment. And yes, that's a bug in Plasma: [1117495.024] -> org_kde_plasma_surface@210.set_position(0, 0) [1117495.036] -> org_kde_kwin_slide_manager@105.unset(wl_surface@88) [1117495.041] -> wl_surface@88.commit() [1117495.051] ->

D8268: Fix placement of KSplash

2017-10-15 Thread Martin Flöser
graesslin added a comment. In https://phabricator.kde.org/D8268#154753, @graesslin wrote: > This needs additional testing for "normal" OSD windows. I'm not sure whether it affects them. It shouldn't or better said: if they are it's a bug somewhere else. So tried this again: it

D8268: Fix placement of KSplash

2017-10-12 Thread Martin Flöser
graesslin added a comment. This needs additional testing for "normal" OSD windows. I'm not sure whether it affects them. It shouldn't or better said: if they are it's a bug somewhere else. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D8268 To: graesslin, #kwin,

D8268: Fix placement of KSplash

2017-10-12 Thread Martin Flöser
graesslin created this revision. graesslin added reviewers: KWin, Plasma. Restricted Application added a project: KWin. Restricted Application added subscribers: kwin, plasma-devel. REVISION SUMMARY In a bug report there was a reference that on multi-screen KSplash is not placed correctly. I