Bug 434615

Summary: On Wayland, quick tile actions and window rules cannot override window's minimum size if needed as on X11
Product: [Plasma] kwin Reporter: Till Schäfer <till2.schaefer>
Component: wayland-genericAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: althafmuhammad141106, boredsquirrel, dmatteo002, indecisiveautomator, kde, khiccy, kyliepct, nate, postix, sam, samuelsumukhreddy, stephan.oostveen, tkk13909
Priority: NOR Keywords: usability, wayland
Version: 5.23.0   
Target Milestone: ---   
Platform: Gentoo Packages   
OS: Linux   
URL: https://codereview.qt-project.org/c/qt/qtwayland/+/524982
See Also: https://bugs.kde.org/show_bug.cgi?id=395476
https://bugs.kde.org/show_bug.cgi?id=454827
Latest Commit: Version Fixed In: 6.0.2 with Qt 6.7 or later
Sentry Crash Report:
Attachments: screenshot of quick tiled systemsettings

Description Till Schäfer 2021-03-19 09:42:25 UTC
Created attachment 136840 [details]
screenshot of quick tiled systemsettings

When I quick tile a window with minimum width larger than the half screen width (e.g., systemsettings on my laptop) to the right it is moved out of the screen area. See the attaches screenshot for an example. It only happens on quick tiling to the right, as the left side of the window seems to be the anchor point. Thus, when quick tiling to the left, the window just occupies more than half of the screen width. 

This only happens on wayland. On X11, it seems like kwin is enforcing a smaller width of the window, regardless or the minimum width that is achievable by resizing the window by dragging the window edge.

Possible solution might be: 
1. set the anchor point to the right side of the window when quick tiling to the right.
2. force a smaller width like in X11. However, this might brake some applications layout. So it seems suboptimal to me.

I have not tested this problem for the vertical direction (min height, top-down-tiling). However, when fixing this issue, one should apply the solution in analogously.  

I also think that the minimum width of the systemsettings should be reduced such that it can fit half of a 1920x1080 resolution, but I will open another bug report for that. 

Operating System: Gentoo Linux
KDE Plasma Version: 5.21.3
KDE Frameworks Version: 5.80.0
Qt Version: 5.15.2
Kernel Version: 5.11.7-gentoo
OS Type: 64-bit
Graphics Platform: Wayland
Processors: 8 × Intel® Core™ i7-4810MQ CPU @ 2.80GHz
Memory: 15.5 GiB of RAM
Graphics Processor: Mesa DRI Intel® HD Graphics 4600
Comment 1 Till Schäfer 2021-03-19 09:53:37 UTC
Bug 434616 - systemsettings minimum width is too large
Comment 2 Nate Graham 2021-03-19 19:35:17 UTC
Confirmed. Looks like kwin_wayland actually respect's a window's minimum size, whereas kwin_x11 ignores it if it would need to be made smaller to fit in the tiles area.

Arguably this is intentional, but I can see how it might be undesirable under certain circumstances.
Comment 3 Till Schäfer 2022-05-23 18:27:27 UTC
*** Bug 447731 has been marked as a duplicate of this bug. ***
Comment 4 Nate Graham 2022-05-23 19:35:08 UTC
*** Bug 454127 has been marked as a duplicate of this bug. ***
Comment 5 Nate Graham 2022-06-21 18:14:42 UTC
*** Bug 455665 has been marked as a duplicate of this bug. ***
Comment 6 Nate Graham 2022-06-21 18:26:35 UTC
*** Bug 455562 has been marked as a duplicate of this bug. ***
Comment 7 Nate Graham 2022-12-01 22:48:18 UTC
This can cause a window to end up in immovable state; when I quarter-tile System Settings to the top-right or top-left corner, for example its titlebar moves out of view so it can only be moves away by Meta+dragging or using keyboard shortcuts.
Comment 8 Nate Graham 2022-12-05 19:59:17 UTC
*** Bug 462668 has been marked as a duplicate of this bug. ***
Comment 9 Nate Graham 2023-03-02 21:35:04 UTC
*** Bug 466717 has been marked as a duplicate of this bug. ***
Comment 10 Nate Graham 2023-12-12 17:22:57 UTC
*** Bug 478366 has been marked as a duplicate of this bug. ***
Comment 11 David Edmundson 2023-12-13 11:42:19 UTC
I filed https://codereview.qt-project.org/c/qt/qtwayland/+/524982 which removes a place where constrains are upheld.

We still have an issue kwin side that window rules for minimum sizes do not appear to be working.
Comment 12 David Edmundson 2023-12-13 13:11:02 UTC
Edit, we don't have an issue our side, we just had another Qt issue which I think I've fixed
Comment 13 Kye 2024-03-08 20:47:36 UTC
Just upgraded to plasma 6.0.1, this issue seems to have returned. 

Tested with `spotify --ozone-platform=wayland`. Works under Xwayland as on X11
Comment 14 Nate Graham 2024-03-08 22:54:13 UTC
Indeed, and it's fixed again in 6.0.2. :) 

In general please open new bug reports rather than re-opening old ones, as it's quite common for issues with the same outward issues to have different root causes.
Comment 15 Nate Graham 2024-03-11 23:32:04 UTC
*** Bug 483081 has been marked as a duplicate of this bug. ***
Comment 16 Kye 2024-03-15 03:32:13 UTC
(In reply to Nate Graham from comment #14)
> Indeed, and it's fixed again in 6.0.2. :) 
> 
> In general please open new bug reports rather than re-opening old ones, as
> it's quite common for issues with the same outward issues to have different
> root causes.

On plasma 6.0.2 Wayland right now and this issue still seems to persist?
Comment 17 Nate Graham 2024-03-15 04:45:32 UTC
Just tested again and I can confirm that it's still broken for me too. However looking at the Qt patch, it only made it into Qt 6.7, which isn't released yet. So it looks like this is in fact fixed, just not yet for those of us using released versions of Qt.
Comment 18 Kye 2024-04-19 22:00:19 UTC
I am on Qt 6.7 and Plasma 6.0.4 now and am still unable to remove the minimum size restriction on `spotify --ozone-platform=wayland` with window rules. Am I doing something incorrectly?