Bug 419540 - Button to assign a keyboard shortcut shows pressed shortcut twice if it includes Meta key
Summary: Button to assign a keyboard shortcut shows pressed shortcut twice if it inclu...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: general (show other bugs)
Version: master
Platform: Neon Linux
: NOR normal
Target Milestone: 1.0
Assignee: David Edmundson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-04-02 13:43 UTC by Patrick Silva
Modified: 2020-04-09 13:14 UTC (History)
2 users (show)

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


Attachments
screenshot (44.23 KB, image/png)
2020-04-02 13:43 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
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