345411 2015-03-22 04:41:57 +0000 Secondary shortcuts are not persisted correctly 2015-06-08 20:20:07 +0000 1 1 3 Frameworks and Libraries frameworks-kxmlgui general 5.0.0 Arch Linux Linux RESOLVED FIXED NOR major --- 0 kde kdelibs-bugs-null aroth kde linds.r msx peter tim http://commits.kde.org/kxmlgui/c7b1b1b89e388779356a5dcde291d6e6eac7705b 5.10.0 0 oldest_to_newest 1506709 0 kde 2015-03-22 04:41:57 +0000 Changing the secondary shortcut for an action which has two results in the primary shortcut being cleared. Reproducible: Always Steps to Reproduce: 1. Open the Configure Shortcuts dialog in an affected application (I've reproduced this in Kate and Konsole). 2. Find an action which by default has two key bindings (for example, Kate's Cut action has ctrl-X and shift-delete). 3. Change or remove the secondary binding. 4. Restart the application and open the Configure Shortcuts dialog again. Actual Results: The primary shortcut is cleared (set to Custom, with the custom binding set to None). The secondary shortcut is reset to its default. Expected Results: The key bindings I set are saved and restored the next time I run the application. By experimenting with strace I've found that this is (as far as I can tell) stored in ~/.local/share/kxmlgui5/katepart/katepart5ui.rc. The following is added to that file after making the change: <ActionProperties scheme="Default"> <Action shortcut="Ctrl+X; " name="edit_cut"/> </ActionProperties> However, Kate's UI still displays "Custom: None" and "Default: Shift+Del". 1512567 1 peter 2015-04-21 05:35:41 +0000 I'm seeing this too, on Kubuntu 15.04. (KDE 5.9.0) libkf5xmlgui5:amd64 5.9.0-0ubuntu1 I hadn't figured out yet that it was only multiple keybindings that were causing problems, so that's a help to be able to at least pick one. 1513692 2 linds.r 2015-04-26 13:04:53 +0000 Confirmed on git master. Believe this bug is the cause behind the Konsole bugs: https://bugs.kde.org/show_bug.cgi?id=340803 https://bugs.kde.org/show_bug.cgi?id=339243 https://bugs.kde.org/show_bug.cgi?id=337131 1514208 3 aroth 2015-04-28 05:01:17 +0000 I build kxmlgui with the patch from https://git.reviewboard.kde.org/r/123508/ and it works fine. Thanks Lindsay to track this down. 1515311 4 linds.r 2015-05-02 09:04:57 +0000 Git commit c7b1b1b89e388779356a5dcde291d6e6eac7705b by Lindsay Roberts. Committed on 02/05/2015 at 08:57. Pushed by roberts into branch 'master'. Fix shortcuts when secondary shortcut set. If a user defined secondary ("alternate") shortcut is set, an issue in KXMLGUIFactory causes reloading of both primary and secondary from the RC file's DOM to fail, resulting in no effective shortcuts available for the action. Fix by checking for the correct type in the shortcut property for QActions. REVIEW: 123508 M +36 -0 autotests/kxmlgui_unittest.cpp M +1 -0 autotests/kxmlgui_unittest.h M +3 -2 src/kxmlguifactory.cpp http://commits.kde.org/kxmlgui/c7b1b1b89e388779356a5dcde291d6e6eac7705b 1515314 5 linds.r 2015-05-02 09:13:50 +0000 *** Bug 339243 has been marked as a duplicate of this bug. *** 1515316 6 linds.r 2015-05-02 09:15:34 +0000 *** Bug 340803 has been marked as a duplicate of this bug. *** 1524947 7 christoph 2015-06-08 20:20:07 +0000 *** Bug 346950 has been marked as a duplicate of this bug. ***