Bug 352254

Summary:	Windows flying around when unmaximized (multi-monitor setup)
Product:	[Plasma] kwin	Reporter:	John Lindgren <john>
Component:	effects-window-management	Assignee:	KWin default assignee <kwin-bugs-null>
Status:	RESOLVED UNMAINTAINED
Severity:	minor	CC:	cpigat242, postix
Priority:	NOR
Version First Reported In:	5.4.0
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
URL:	https://bugs.kde.org/attachment.cgi?id=92787
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description John Lindgren 2015-09-04 05:12:56 UTC

There is a minor problem still remaining with windows maximized on multiple monitors.  You can see it occurring several times in the video attached to bug #348121.  Sometimes, there is a weird animation when restoring the window to normal size, where the window flies in from the side of the screen.  (I'm assuming this is not intentional.)

Reproducible: Always

Steps to Reproduce:
1. Drag a window to the top of the second monitor to maximize it.
2. Drag it downward to unmaximize it.

Actual Results:  
The window flies in rapidly from the right-hand edge of the screen.

Expected Results:  
The window should animate from its current maximized position.

Comment 1 Marco Martin 2016-03-04 17:19:47 UTC

Git commit 595c5937c9a1e13bdeed2759034ed66d8c89d614 by Marco Martin.
Committed on 04/03/2016 at 17:19.
Pushed by mart into branch 'master'.

use a global static for animation ids

using stack adresses makes it possible (even tough not frequent)
for two animations (onle just killed, one just created) to have
the same id, causing scripts to be potentially really confused.
this replaces the id with a global counter, and the
"flying tooltips" bug is gone.

REVIEW:127276
Related: bug 360068

M  +2    -0    libkwineffects/anidata_p.h
M  +8    -4    libkwineffects/kwinanimationeffect.cpp

http://commits.kde.org/kwin/595c5937c9a1e13bdeed2759034ed66d8c89d614

Comment 2 Martin Flöser 2016-03-23 11:06:16 UTC

Git commit a9fad7396e30a380a27090538e7e66dec6984c52 by Martin Gräßlin, on behalf of Marco Martin.
Committed on 23/03/2016 at 11:03.
Pushed by graesslin into branch 'Plasma/5.6'.

use a global static for animation ids

using stack adresses makes it possible (even tough not frequent)
for two animations (onle just killed, one just created) to have
the same id, causing scripts to be potentially really confused.
this replaces the id with a global counter, and the
"flying tooltips" bug is gone.

REVIEW:127276
Related: bug 360068

M  +2    -0    libkwineffects/anidata_p.h
M  +8    -4    libkwineffects/kwinanimationeffect.cpp

http://commits.kde.org/kwin/a9fad7396e30a380a27090538e7e66dec6984c52

Comment 3 David Edmundson 2023-09-06 10:38:37 UTC

This bug was reported against an outdated version of KWin. We have made many changes since the. 
If the issue persists in newer versions can you reopen the bug report updating the version number.