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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.