D24542: explicitly place fullscreen windows over fullscreen
Marco Martin <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
mart created this revision. mart added reviewers: KWin, Plasma. Herald added a project: KWin. Herald added a subscriber: kwin. mart requested review of this revision. REVISION SUMMARY on X11, setFullScreen is always called after Placement::place() so they always have the correct geometry. on wayland, the window if is shown directly as fullscren, is set fullscreen in init() then place() is evecuted, potentially moving it to a wrong position and potentially even size (which happens with maximiziong placement strategy) so instead of place() the client needs to be explicitly set at fullscreen geometry TEST PLAN fullscreen windows always appear with the proper geometry autotests still pass REPOSITORY R108 KWin BRANCH phab/fullscreen REVISION DETAIL https://phabricator.kde.org/D24542 AFFECTED FILES xdgshellclient.cpp To: mart, #kwin, #plasma Cc: kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart