Bug 484999 - [Overview] MMB-clicking on added Virtual Desktop should not remove it
Summary: [Overview] MMB-clicking on added Virtual Desktop should not remove it
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: effects-overview (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2024-04-03 21:35 UTC by bugsie
Modified: 2024-04-13 14:06 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.1


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description bugsie 2024-04-03 21:35:43 UTC
SUMMARY

In Overview, Middle-mouse-button clicking any extra Virtual Desktop should not remove it. There's a trashbin icon already for that, which should (also) require confirmation. 

Virtual Desktops are many times set up "permanently". By deleting them that easily with MMB, "Window Rules" set for them are being destroyed as well, which are a pain to set up again.

STEPS TO REPRODUCE
1. Hit "Meta+W" to go into Overview. 
2. Create a new Virtual Desktop with the + button.
3. MMB-click the newly created Virtual Desktop.

OBSERVED RESULT
Virtual Desktop is being removed.

EXPECTED RESULT
Nothing.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 6.0.3
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.3
Kernel Version: 6.8.3-AMD-znver3 (64-bit)
Graphics Platform: Wayland
Comment 1 Bug Janitor Service 2024-04-11 13:37:51 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/5600
Comment 2 Nate Graham 2024-04-12 14:38:29 UTC
Git commit 99324c12fd6b689809aad77ea3cb852d89982273 by Nate Graham.
Committed on 12/04/2024 at 13:25.
Pushed by ngraham into branch 'master'.

plugins/overview: remove middle click to close virtual desktop

This is a hidden destructive action[1] in a context where users are not
primed to expect it. It's also largely unnecessary since the delegates
already show a delete button on hover, and deleting virtual desktops
isn't such a common task that it benefits from an accelerator. Let's
remove it for safety.

[1] It's destructive because user-created state on the deleted virtual
desktop is lost--with no confirmation or undo, to boot.
FIXED-IN: 6.1

M  +2    -9    src/plugins/overview/qml/DesktopBar.qml

https://invent.kde.org/plasma/kwin/-/commit/99324c12fd6b689809aad77ea3cb852d89982273