Bug 489084

Summary:	Moving a shaded window loses or changes the window height
Product:	[Plasma] kwin	Reporter:	Nick <nick>
Component:	Window shading	Assignee:	KWin default assignee <kwin-bugs-null>
Status:	ASSIGNED ---
Severity:	normal	CC:	7eggert, kdedev, luca.pedrielli, mguiddy, nate, rpbikker
Priority:	NOR	Keywords:	regression
Version First Reported In:	6.1.0
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Nick 2024-06-24 03:46:23 UTC

SUMMARY
If you shade a window (by double-clicking the title bar), then move it while it's shaded, then attempt to unshade it, it will either now be only 1 border-width-tall, or maybe slightly shorter than before--but won't be the correct, original height. In the latter case, repeating the sequence (shade, move, unshade) corrupts the height more each time until it's eventually 0.

STEPS TO REPRODUCE
1. open a window.
2. double-click the title bar to shade it
3. drag/move the shaded window (visible title bar)
4. double-click the title bar to unshade it

OBSERVED RESULT
It's not the correct, original window height that it had before it was shaded.

EXPECTED RESULT
Window height should be preserved between shade and unshade operations, regardless of if/where the title bar is moved around.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 6.1.0
KDE Frameworks Version: 6.3.0
Qt Version: 6.7.2
X11 session (not wayland)

ADDITIONAL INFORMATION

Comment 1 Reinier 2024-06-24 17:21:00 UTC

Yup, seeing this too. Exact same software versions. Didn't see this with v6.0.5.

Comment 2 Nate Graham 2024-06-25 15:30:55 UTC

Note that window shading is not actively maintained as it cannot be made to work cleanly on Wayland. So unfortunately I would not expect quick resolution here.

Comment 3 Nick 2024-06-25 15:49:35 UTC

(In reply to Nate Graham from comment #2)
> Note that window shading is not actively maintained as it cannot be made to
> work cleanly on Wayland. So unfortunately I would not expect quick
> resolution here.

I'm not using wayland.
This is a rather serious regression.
New features shouldn't break old, previously-working things...

Comment 4 Bodo Eggert 2024-07-26 13:04:02 UTC

If the new version neither works for X11 nor Wayland, why is it in the release version at all? Wasn't it tested?

AFAIK it's due to a workaround for Wayland not offering essential features, so please annoy the Wayland maintainers, not the users.

Comment 5 Nick 2024-10-12 15:32:14 UTC

Still broken in 6.2.0

Comment 6 TraceyC 2024-10-14 20:14:43 UTC

I'm able to reproduce this on git-master (X11)

KDE Plasma Version: 6.2.80
KDE Frameworks Version: 6.7.0
Qt Version: 6.7.3
Kernel Version: 6.10.13-304.current (64-bit)
Graphics Platform: X11

As Nate said, this is not actively maintained so a fix may not be quick. Just adding a data point.

Comment 7 Nate Graham 2024-10-16 23:41:42 UTC

Please don't change the version

Comment 8 Matthew Guiddy 2025-01-14 22:52:10 UTC

To me, it acts the same as this bug that was previously fixed.
https://bugs.kde.org/show_bug.cgi?id=450582
Possibly related?

Comment 9 Matthew Guiddy 2025-01-24 08:08:22 UTC

(In reply to Matthew Guiddy from comment #8)
> To me, it acts the same as this bug that was previously fixed.
> https://bugs.kde.org/show_bug.cgi?id=450582
> Possibly related?

I don't know why I didn't think to try the workaround that worked in the previous bug, but setting window border to "No window border" does make shaded windows movable again without losing their unshaded height.

Comment 10 Bug Janitor Service 2025-08-01 09:43:14 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin-x11/-/merge_requests/66

Comment 11 Bug Janitor Service 2025-08-07 09:32:43 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin-x11/-/merge_requests/69