D8268: Fix placement of KSplash

Martin Flöser noreply at phabricator.kde.org
Thu Oct 12 18:52:58 UTC 2017


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 investigated and noticed that it is an OSD which
  sets an own position. In KWin the events were processed correctly but
  the position was off.
  
  The problem is that KWin has code to correct the position of an OSD when
  it's size changes. This happens also on first damage and then the window
  gets incorrectly placed when the position is set. So honor that the
  position is set.

TEST PLAN
  Restarted the session, ksplash positioned correctly now.

REPOSITORY
  R108 KWin

BRANCH
  ksplash-fix-5.11

REVISION DETAIL
  https://phabricator.kde.org/D8268

AFFECTED FILES
  abstract_client.cpp
  abstract_client.h
  autotests/integration/plasma_surface_test.cpp
  shell_client.h

To: graesslin, #kwin, #plasma
Cc: plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171012/d69803b2/attachment.html>


More information about the Plasma-devel mailing list