Bug 477657 - Lost Meta+V shortcut in transition to Plasma 6
Summary: Lost Meta+V shortcut in transition to Plasma 6
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: general (show other bugs)
Version: 5.27.80
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords: qt6, regression
Depends on:
Blocks:
 
Reported: 2023-11-28 08:44 UTC by Eric Armbruster
Modified: 2024-02-16 12:19 UTC (History)
3 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Armbruster 2023-11-28 08:44:24 UTC
SUMMARY
***
NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols.
See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***


STEPS TO REPRODUCE
1. Upgrade to 6
2. 
3. 

OBSERVED RESULT
Keyboard shortcut for show clipboard items at mouse position unset after upgrade

When trying to set the shortcut manually, I get that there is a conflict with already an existing action (though it does not tell me which one, which is another known bug).

If I were to guess the problem is buried somewhere in a framework not being able to deal with the action name change done here https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3082

EXPECTED RESULT
shortcut is migrated correctly
Operating System: Arch Linux 
KDE Plasma Version: 5.27.80
KDE Frameworks Version: 5.245.0
Qt Version: 6.6.0
Kernel Version: 6.6.2-arch1-1 (64-bit)
Graphics Platform: Wayland
Processors: 16 × AMD Ryzen 7 5700U with Radeon Graphics
Memory: 38.0 GiB of RAM
Graphics Processor: AMD Radeon Graphics
Manufacturer: LENOVO
Product Name: 20Y7CTO1WW
System Version: ThinkPad E14 Gen 3

ADDITIONAL INFORMATION
Please assign to correct product
Comment 1 David Redondo 2023-11-28 10:10:21 UTC
Do you have multiple entries if you search for "Meta+V" in shortcut settings?

I have Klipper and plasmashell, only plasmashell works
Comment 2 Eric Armbruster 2023-11-28 11:44:49 UTC
I had two entries with meta+v
kwin and plasmashell

I now tried deleting them both. Does not help. 

Sidenote: I tried resetting to default as I would expect there to be some default shortcut with meta+v to appear again after pressing default button, but that seems bugged too :(
Comment 3 Eric Armbruster 2023-11-28 11:45:33 UTC
I have still set the meta+v shortcut in the klipper applet though, but still no result when pressing meta+v
Comment 4 Eric Armbruster 2023-11-28 11:50:24 UTC
To be more clear, in the shortcuts kcm when i search now for meta+v no result is shown. but i have set meta+v in the klipper applet
Comment 5 David Redondo 2023-11-28 12:31:07 UTC
Hmm, in the applet is it set as "Global"?
Comment 6 Eric Armbruster 2023-11-28 14:40:35 UTC
Heh, thanks! Now it works. Still think there is at least one bug to solve here...
Comment 7 Eric Armbruster 2023-11-28 14:50:38 UTC
The point that I am making is, it was working before and should not require manual intervention in the transition to 6. Though I don't quite know whether this whole issue was caused by me having configured something weirdly before.
Comment 8 David Redondo 2023-11-28 15:59:33 UTC
Yes it should. I dont know why it broke for you. A shortcut being in  KWin  seems to be a hint and I can see it on my machine as well existing.

I checked a plasma 5.27 ISO and there the shortcut is inside plasmashell category (the one that works for me on Plasma 6), so at least they match which is good.
Comment 9 Nate Graham 2023-11-28 20:50:42 UTC
Cannot reproduce, FWIW.
Comment 10 Fushan Wen 2023-11-30 01:12:20 UTC
Can reproduce. kwin duplicates shortcuts which causes the problem.
Comment 11 Vlad Zahorodnii 2024-02-16 09:37:52 UTC
(In reply to Fushan Wen from comment #10)
> Can reproduce. kwin duplicates shortcuts which causes the problem.

Can you provide more details? kwin doesn't register this shortcut afaik
Comment 12 David Redondo 2024-02-16 10:17:21 UTC
I think it could happen if you somehow loaded the clipboard data engine inside kwin?
Comment 13 David Redondo 2024-02-16 10:27:27 UTC
Checking out my kglobalshortcutsrc I see actually more shortcuts which belong to applets in the KWin group


activate task manager entry 1..10
activate widget $n
cycle-panels=none,Meta+Alt+P,Tastaturfokus zwischen Kontrollleisten umschalten
manage activities=none,Meta+Q,Aktivitätenwechsler anzeigen
next activity=none,Meta+A,Zwischen Aktivitäten wechseln
previous activity=Meta+Shift+A,Meta+Shift+A,Zwischen Aktivitäten wechseln (Gegenrichtung)
toggle do not disturb=none,none,Nicht stören umschalten
Comment 14 David Redondo 2024-02-16 10:33:00 UTC
Not only applets but plasmashell as a whole.
Comment 15 David Redondo 2024-02-16 10:51:12 UTC
I cleaned my kglobalshortcutsrc of the offending entries, and saw that two config updates were applied

update_info=kwin.upd:change-activities-overview-shortcuts


Running ./kwin-6.0-delete-desktop-switching-shortcuts adds the wrong shortcuts to the kwin group
Comment 16 David Redondo 2024-02-16 12:19:22 UTC
My testing was wrong, since the false entries where still in memory of the daemon.

I think what actually happened that we had
https://invent.kde.org/plasma/kwin/-/blob/028dd552cfb9d826b40b9620d869c98d2aa3dca3/kconf_update/kwin.upd with

Version=5
Group=plasmashell,kwin

this  made kf5 kconf_update (we had both kded6 and kded5  running for a time) copy the plasmashell group to KWin.

The offending update was removed and the version bumped, so it should not happen anymore on upgrade
https://invent.kde.org/plasma/kwin/-/commit/25150633edfb332bc765e6bf3930c7ba9fc0a172
https://invent.kde.org/plasma/kwin/-/commit/540dff30e7f014809fa25e9889f6174dec285f53