Bug 450487

Summary: Window close shortcut should close focused window in Present Windows UI
Product: [Plasma] kwin Reporter: Lukas Sabota <lukas>
Component: effects-present-windowsAssignee: KWin default assignee <kwin-bugs-null>
Status: CONFIRMED ---    
Severity: wishlist CC: nate
Priority: NOR    
Version: 5.24.1   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In:

Description Lukas Sabota 2022-02-17 21:40:20 UTC
SUMMARY
In the present windows UI, there is currently no way (that I am aware of) to close a focused window using the keyboard.  In order to close a window while in the Present Windows UI, the user must press the red close icon associated with the window they would like to close.
I think it would be a usability improvement to be able to close windows using the configured "Close Window" keyboard shortcut.  This would allow a user to be able to get a quick visual overview of the windows they have open on their desktop (or all desktops) and quickly be able to close unnecessary windows using a keyboard shortcut they are already familiar with.


STEPS TO REPRODUCE
1. Open the present windows UI using the configured "Expose" or "ExposeAll" keyboard shortcut (or the hot corner).
2. Highlight a window.
3. Press the configured "Close Window" keyboard shortcut (Alt+f4 by default)

OBSERVED RESULT
The Close Window keyboard shortcut has no effect while the user is in the Present Windows UI.
The user must use the mouse to close windows while in the Present Windows UI

EXPECTED RESULT


SOFTWARE/OS VERSIONS
Operating System: Arch Linux
KDE Plasma Version: 5.24.1
KDE Frameworks Version: 5.91.0
Qt Version: 5.15.2
Kernel Version: 5.16.9-arch1-1 (64-bit)
Graphics Platform: X11

ADDITIONAL INFORMATION
Thank you to everyone who has contributed to this awesome desktop environment!
Comment 1 Marco Martin 2022-05-06 11:37:34 UTC
Git commit ae1937badc097a864ecfc62e79e7a3899c3be572 by Marco Martin.
Committed on 06/05/2022 at 11:37.
Pushed by mart into branch 'master'.

Remove completely present windows

since it has been replaced by windowview, and Desktop Grid
is ported as well, remove present windows which is effectively dead code
now
Related: bug 447001, bug 362844, bug 453426, bug 374481, bug 185381, bug 413342, bug 451150, bug 283333, bug 315314, bug 397500, bug 321236, bug 436572, bug 335782

M  +0    -1    src/effects/CMakeLists.txt
D  +0    -45   src/effects/presentwindows/CMakeLists.txt
D  +0    -17   src/effects/presentwindows/main.cpp
D  +0    -18   src/effects/presentwindows/main.qml
D  +0    -75   src/effects/presentwindows/metadata.json
D  +0    -2054 src/effects/presentwindows/presentwindows.cpp
D  +0    -343  src/effects/presentwindows/presentwindows.h
D  +0    -51   src/effects/presentwindows/presentwindows.kcfg
D  +0    -104  src/effects/presentwindows/presentwindows_config.cpp
D  +0    -46   src/effects/presentwindows/presentwindows_config.h
D  +0    -487  src/effects/presentwindows/presentwindows_config.ui
D  +0    -36   src/effects/presentwindows/presentwindows_proxy.cpp
D  +0    -35   src/effects/presentwindows/presentwindows_proxy.h
D  +0    -6    src/effects/presentwindows/presentwindowsconfig.kcfgc

https://invent.kde.org/plasma/kwin/commit/ae1937badc097a864ecfc62e79e7a3899c3be572
Comment 2 Nate Graham 2022-05-06 13:19:47 UTC
Removing the old code doesn't make wishlist requests invalid for the new version (which has the same name, so they are the same thing from the user's perspective); re-opening.