Bug 359423

Summary:	Quick tiling fails on inner screen borders (pre to special handling of inner borders)
Product:	[Plasma] kwin	Reporter:	Jonathan Kolberg <j_kolberg11>
Component:	xrandr	Assignee:	KWin default assignee <kwin-bugs-null>
Status:	RESOLVED WORKSFORME
Severity:	normal
Priority:	NOR
Version First Reported In:	5.4.3
Target Milestone:	---
Platform:	Kubuntu
OS:	Linux
Latest Commit:		Version Fixed In:	5.5
Sentry Crash Report:
Attachments:	My multiscreen info

Description Jonathan Kolberg 2016-02-15 10:07:05 UTC

If I have at least two monitors configured to be next to each other. Snapping a window on each monitor does not work if snapped to the side where the other monitor is.

Reproducible: Always

Steps to Reproduce:
1. Monitor 1 is left
2. Monitor 2 is right
3. Dragg a window on monitor 1 to the right side of monitor 1

Actual Results:  
Window is moved and not snapped

Expected Results:  
Window snapps to the right half of monitor 1

Comment 1 Thomas Lübking 2016-02-15 13:57:34 UTC

Please specify whether by "snapping" you refer to actual snapping (ie. slight position corrections) or Quick Tiling (quarter/half sizing the window) and attach the output of "qdbus org.kde.KWin /KWin supportInformation" (taken on the multiscreen setup)

Comment 2 Jonathan Kolberg 2016-02-15 14:02:48 UTC

I meant the Quick Tiling.

Comment 3 Jonathan Kolberg 2016-02-15 14:04:19 UTC

Created attachment 97231 [details]
My multiscreen info

Comment 4 Thomas Lübking 2016-02-15 15:50:46 UTC

Screen 0:
---------
Geometry: 0,1152,1366x768

Screen 1:
---------
Geometry: 1366,0,1080x1920

Does it also fail if you move screen 0 to 0,0, ie. have them top-aligned?

Comment 5 Jonathan Kolberg 2016-02-15 16:48:34 UTC

I have to check that tomorrow, since the external monitor is at my university :)

Comment 6 Jonathan Kolberg 2016-02-16 08:36:26 UTC

Aligning on top work, but it should also work if not aligned on top

Comment 7 Thomas Lübking 2016-02-16 11:48:49 UTC

Yes, certainly should.
Does here (git master) - also tried disjunct screens (works) and (weird) overlapping setups (work weirdly ;-)

I don't see at hand why it would fail in the 5.4.x code, but it saw some rewrite (commit 3a1f11d21347032f7d1b8fd43d30ee7d3ce5ede0 and follow up patch), so it might indeed have fixed "by the way".
=> any chance to test KWin 5.5.x ?

Comment 8 Jonathan Kolberg 2016-02-16 12:33:54 UTC

Tested in kwin 5.5.4 and it's fixed there.

Comment 9 Thomas Lübking 2016-02-16 12:36:16 UTC

See, I can fix bugs on the fly that I'm not even aware of ;-)

Many thanks for the status update.