Bug 466120

Summary: Window Class mode invisibly includes all windows, so you can access them using the keyboard
Product: [Plasma] kwin Reporter: nyanpasu64 <nyanpasu64>
Component: effects-present-windowsAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: nate
Priority: HI Keywords: usability
Version: 5.27.0   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.27.2
Sentry Crash Report:

Description nyanpasu64 2023-02-20 04:41:10 UTC
SUMMARY
When I set "Icons-only Task Manager" grouped tasks to "Shows large window previews", and click a grouped task, arrow keys can select windows not visible and not from the current app.

STEPS TO REPRODUCE
1. In "Icons-only Task Manager Settings", in Behavior tab, set "Clicking grouped task: Shows large window previews" and apply.
2. Open multiple windows under an app, and press Super+Number or click the app's icon. This pops up full-screen previews of the app's windows.
3. Use the arrow keys to navigate between apps.

OBSERVED RESULT
The focus disappears. Pressing Enter opens a window not from the app you clicked.

EXPECTED RESULT
The blue focus rectangle only travels between visible windows, coming from the app you clicked.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 5.27.0
KDE Frameworks Version: 5.103.0
Qt Version: 5.15.8
Kernel Version: 6.1.12-zen1-1-zen (64-bit)
Graphics Platform: X11
Processors: 12 × AMD Ryzen 5 5600X 6-Core Processor
Memory: 15.5 GiB of RAM
Graphics Processor: AMD Radeon RX 570 Series
Manufacturer: Gigabyte Technology Co., Ltd.
Product Name: B550M DS3H

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2023-02-22 02:05:27 UTC
OMG
Comment 2 Bug Janitor Service 2023-02-22 08:23:06 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/3660
Comment 3 Vlad Zahorodnii 2023-02-22 13:22:43 UTC
Git commit bc0502b3cac939c5e75e63385dc9b4e2f32e9925 by Vlad Zahorodnii.
Committed on 22/02/2023 at 11:53.
Pushed by vladz into branch 'master'.

effects: Fix selecting hidden windows using keyboard

WindowHeapDelegate used to have hidden property. It was renamed to
activeHidden, but WindowHeap has not been adjusted to the new name.

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

https://invent.kde.org/plasma/kwin/commit/bc0502b3cac939c5e75e63385dc9b4e2f32e9925
Comment 4 Vlad Zahorodnii 2023-02-22 14:41:20 UTC
Git commit 7ea6506d61cb741e18e944bbf74b485c439b9631 by Vlad Zahorodnii.
Committed on 22/02/2023 at 13:32.
Pushed by vladz into branch 'Plasma/5.27'.

effects: Fix selecting hidden windows using keyboard

WindowHeapDelegate used to have hidden property. It was renamed to
activeHidden, but WindowHeap has not been adjusted to the new name.


(cherry picked from commit bc0502b3cac939c5e75e63385dc9b4e2f32e9925)

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

https://invent.kde.org/plasma/kwin/commit/7ea6506d61cb741e18e944bbf74b485c439b9631