Bug 501731 - "Quick Tile Window to" actions need a second step to move to correct if target window is already tiled
Summary: "Quick Tile Window to" actions need a second step to move to correct if targe...
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: Quick Tiling (show other bugs)
Version: 6.3.3
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2025-03-19 05:47 UTC by Christoph Dunschen
Modified: 2025-03-21 19:05 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.3.4
Sentry Crash Report:


Attachments
My config (164.87 KB, image/jpeg)
2025-03-19 05:47 UTC, Christoph Dunschen
Details
My configuration (164.87 KB, image/jpeg)
2025-03-19 05:47 UTC, Christoph Dunschen
Details
Starting positon of window (581.51 KB, image/jpeg)
2025-03-19 05:50 UTC, Christoph Dunschen
Details
When I press first time Meta+F4 (516.99 KB, image/jpeg)
2025-03-19 05:51 UTC, Christoph Dunschen
Details
When I press the second time Meta+F4 (454.14 KB, image/jpeg)
2025-03-19 05:51 UTC, Christoph Dunschen
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Christoph Dunschen 2025-03-19 05:47:20 UTC
Created attachment 179563 [details]
My config

***
"Quick Tile Window to" function need a second step to move to correct position
***

SUMMARY
With one of the last 6.3 Plasma updates, the "Quick Tile Window to " function is broken.

STEPS TO REPRODUCE
1. See my config at attachment "KWin01.jpg" (overwrite setting if asked...)
2. When I have a window in "Top Left" position and press Meta+F4, result is in the middle of screen
3. Pressing Meta+F4 again, the correct position "Bottom Right" is found

OBSERVED RESULT
Need to press Meta+F4 two times to reach correct position

EXPECTED RESULT
Press Meta+F4 only one time

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
(available in the Info Center app, or by running `kinfo` in a terminal window)
Linux/KDE Plasma: 
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Comment 1 Christoph Dunschen 2025-03-19 05:47:58 UTC
Created attachment 179564 [details]
My configuration
Comment 2 Christoph Dunschen 2025-03-19 05:50:42 UTC
Created attachment 179565 [details]
Starting positon of window
Comment 3 Christoph Dunschen 2025-03-19 05:51:15 UTC
Created attachment 179566 [details]
When I press first time Meta+F4
Comment 4 Christoph Dunschen 2025-03-19 05:51:51 UTC
Created attachment 179567 [details]
When I press the second time Meta+F4
Comment 5 Nate Graham 2025-03-19 21:26:44 UTC
Can reproduce. The spatial tiling changes should probably only affect when tiling via arrow keys, not when using the shortcuts that tile directly to exact positions on screen.
Comment 6 Bug Janitor Service 2025-03-20 06:50:32 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/7367
Comment 7 Vlad Zahorodnii 2025-03-21 06:24:38 UTC
Git commit 94048faba57ad8d51be7db36d51fd2495ecf53b1 by Vlad Zahorodnii.
Committed on 21/03/2025 at 06:15.
Pushed by vladz into branch 'master'.

Combine tile modes when tiling via arrow keys

It takes some effort for the human processing unit (aka brain) to combine
quarter tile modes with other tile modes.

This change allows only half tile modes to be combined with other modes,
it is easier to comprehend and it produces fewer unexpected results.

M  +8    -8    autotests/integration/quick_tiling_test.cpp
M  +13   -1    src/window.cpp

https://invent.kde.org/plasma/kwin/-/commit/94048faba57ad8d51be7db36d51fd2495ecf53b1
Comment 8 Vlad Zahorodnii 2025-03-21 06:50:27 UTC
Git commit b161b5c1f4007495dc41f1de4c497cf6d3c93a2f by Vlad Zahorodnii.
Committed on 21/03/2025 at 06:30.
Pushed by vladz into branch 'Plasma/6.3'.

Combine tile modes when tiling via arrow keys

It takes some effort for the human processing unit (aka brain) to combine
quarter tile modes with other tile modes.

This change allows only half tile modes to be combined with other modes,
it is easier to comprehend and it produces fewer unexpected results.


(cherry picked from commit 94048faba57ad8d51be7db36d51fd2495ecf53b1)

Co-authored-by: Vlad Zahorodnii <vlad.zahorodnii@kde.org>

M  +8    -8    autotests/integration/quick_tiling_test.cpp
M  +13   -1    src/window.cpp

https://invent.kde.org/plasma/kwin/-/commit/b161b5c1f4007495dc41f1de4c497cf6d3c93a2f
Comment 9 Vlad Zahorodnii 2025-03-21 06:51:26 UTC
Git commit d6290dff420f5d5cf7303adafce413a2ea93135d by Vlad Zahorodnii.
Committed on 21/03/2025 at 06:32.
Pushed by vladz into branch 'master'.

Combine tile modes when tiling via arrow keys

It takes some effort for the human processing unit (aka brain) to combine
quarter tile modes with other tile modes.

This change allows only half tile modes to be combined with other modes,
it is easier to comprehend and it produces fewer unexpected results.

M  +8    -8    autotests/integration/quick_tiling_test.cpp
M  +13   -1    src/window.cpp

https://invent.kde.org/plasma/kwin-x11/-/commit/d6290dff420f5d5cf7303adafce413a2ea93135d