--e27c4c1591ac4c03a8682f077cc3a261 Content-Type: text/plain; charset="ascii" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit graesslin created this revision. graesslin added reviewers: KWin, Plasma on Wayland, bshah. Restricted Application added subscribers: kwin, plasma-devel. Restricted Application added projects: Plasma on Wayland, KWin. REVISION SUMMARY The restore geometry gets initially set to 0/0xsize before the placement is done. When going into updateClientArea and then afterwards into AbstractClient::checkWorkspacePosition the geometry restory is used for calculating the new position. This results in windows getting moved to 0/0 when e.g. plugging in a new screen or a panel changes, etc. This change ensures that the restore geometry is set correctly after the first placement. BUG: 366696 REPOSITORY rKWIN KWin BRANCH geometry-restore-after-placement REVISION DETAIL https://phabricator.kde.org/D2627 AFFECTED FILES autotests/integration/plasma_surface_test.cpp shell_client.cpp shell_client.h workspace.cpp EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: graesslin, #kwin, #plasma_on_wayland, bshah Cc: plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas --e27c4c1591ac4c03a8682f077cc3a261 Content-Type: text/html; charset="ascii" Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable
graesslin=20created=20this=20revision. g= raesslin=20added=20reviewers:=20KWin,=20Plasma=20on=20Wayland,=20bshah. Restricted=20Application=20added=20subscribers:=20kwin,=20plasma-devel= . Restricted=20Application=20added=20projects:=20Plasma=20on=20Wayla= nd,=20KWin. | View=20Revision
The=20rest=
ore=20geometry=20gets=20initially=20set=20to=200/0xsize=20before=20the=20pl=
acement
is=20done.=20When=20going=20into=20updateClientArea=20and=20then=20afterwar=
ds=20into
AbstractClient::checkWorkspacePosition=20the=20geometry=20restory=20is=20us=
ed=20for
calculating=20the=20new=20position.=20This=20results=20in=20windows=20getti=
ng=20moved=20to
0/0=20when=20e.g.=20plugging=20in=20a=20new=20screen=20or=20a=20panel=20cha=
nges,=20etc.
This=20change=20ensures=20that=20the=20restore=20geometry=20is=20set=20c=
orrectly=20after=20the
first=20placement.
BUG:=20366696