Bug 457910 - Desktop grid arrow keys navigate between windows not desktops
Summary: Desktop grid arrow keys navigate between windows not desktops
Status: CONFIRMED
Alias: None
Product: kwin
Classification: Plasma
Component: effects-desktop-grid (show other bugs)
Version: 5.25.5
Platform: openSUSE Linux
: NOR normal (vote)
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2022-08-15 11:33 UTC by Ondrej Malek
Modified: 2022-09-27 06:28 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Ondrej Malek 2022-08-15 11:33:15 UTC 
SUMMARY
Since 5.25.0 arrow keys are navigating between windows not workspaces as prior to 5.25. 

STEPS TO REPRODUCE
1. Setup desktop grid effect in settings
2. open it
3. pres arrow key

OBSERVED RESULT
navigates cross windows


EXPECTED RESULT
navigates cross workspaces, selected one is highlighted (preferably with border). When using 4k monitor with 5x4 grid currently selected window border is not visible enough.

SOFTWARE/OS VERSIONS

Operating System: openSUSE Tumbleweed 20220813
KDE Plasma Version: 5.25.4
KDE Frameworks Version: 5.96.0
Qt Version: 5.15.5
Kernel Version: 5.19.0-1-default (64-bit)
Graphics Platform: X11
Processors: 12 × AMD Ryzen 5 5600U with Radeon Graphics
Memory: 38,0 GiB of RAM
Graphics Processor: AMD RENOIR
Manufacturer: LENOVO
Product Name: 20X70042CK
System Version: ThinkPad L15 Gen 2a
Comment 1 Mariusz Glebocki 2022-09-08 13:53:23 UTC 
Caused by a fix in https://bugs.kde.org/show_bug.cgi?id=456068, which adds navigation between windows, not desktops.
Is it intentional decision? I did find navigation between desktops (not windows) useful. There's already Present Windows effect for navigating between all windows.
Comment 2 Ondrej Malek 2022-09-13 21:07:59 UTC 
This is still in 5.25.5
Comment 3 Bug Janitor Service 2022-09-26 21:34:03 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/3003
Comment 4 Vlad Zahorodnii 2022-09-27 06:26:45 UTC 
Git commit 105af544c9069f2ccbcb80fc460e7c9dba2a26f5 by Vlad Zahorodnii, on behalf of David Edmundson.
Committed on 27/09/2022 at 06:26.
Pushed by vladz into branch 'master'.

Don't accept  space and enter key events in empty WindowHeaps

This fixes being able to select an empty desktop.

M  +1    -1    src/effects/private/qml/WindowHeap.qml

https://invent.kde.org/plasma/kwin/commit/105af544c9069f2ccbcb80fc460e7c9dba2a26f5
Comment 5 Vlad Zahorodnii 2022-09-27 06:28:49 UTC 
Git commit 893d58eba7800f2fa6f05c6e630ed419ed4ce083 by Vlad Zahorodnii, on behalf of David Edmundson.
Committed on 27/09/2022 at 06:28.
Pushed by vladz into branch 'Plasma/5.26'.

Don't accept  space and enter key events in empty WindowHeaps

This fixes being able to select an empty desktop.


(cherry picked from commit 105af544c9069f2ccbcb80fc460e7c9dba2a26f5)

M  +1    -1    src/effects/private/qml/WindowHeap.qml

https://invent.kde.org/plasma/kwin/commit/893d58eba7800f2fa6f05c6e630ed419ed4ce083