Bug 419540

Summary: Button to assign a keyboard shortcut shows pressed shortcut twice if it includes Meta key
Product: [Plasma] plasmashell Reporter: Patrick Silva <bugseforuns>
Component: generalAssignee: David Edmundson <kde>
Status: RESOLVED FIXED    
Severity: normal CC: kde, plasma-bugs-null
Priority: NOR    
Version First Reported In: master   
Target Milestone: 1.0   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: screenshot

Description Patrick Silva 2020-04-02 13:43:50 UTC
Created attachment 127191 [details]
screenshot

SUMMARY
This issue affects both Arch Linux (Plasma 5.18.4) and Neon unstable edition,
but I can't reproduce consistently. If you fail to reproduce, repeat the steps below trying to assign a different shortcut that also includes Meta key.

STEPS TO REPRODUCE
1. open the settings window of an applet in systray (sound, for example)
2. in "keyboard shortcuts" section, try to assign any shortcut including Meta
key and that is not in use yet: as we can see in the attached screenshot, button to assign the shortcut shows the pressed shortcut twice.
3. click on "Cancel" button of "<applet> settings" window
4. try to assign the same shortcut to the same applet again: now the button to assign the shortcut shows the pressed shortcut once as expected

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.18.80
KDE Frameworks Version: 5.69.0
Qt Version: 5.14.1


ADDITIONAL INFORMATION
Despite the pressed shortcut is duplicated after the step 2, it works as expected if confirmed with click on "Ok" button of "<applet> settings" window.
Comment 1 Kai Uwe Broulik 2020-04-02 13:44:53 UTC
Can you check if https://phabricator.kde.org/D26016 fixes that?
Comment 2 Patrick Silva 2020-04-02 15:39:54 UTC
Done. Your patch fixes the bug on both Arch and Neon unstable.
Comment 3 Kai Uwe Broulik 2020-04-09 13:14:54 UTC
Patches pushed, thanks