! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
keyboard settings panel ignores state of "startup-notification" key when rese...
Status:
RESOLVED: FIXED
Product:
Libxfce4ui
Component:
General

Comments

Description Alistair Buxton 2014-01-13 18:48:33 CET
To reproduce:

1. xfconf-query -c xfce4-keyboard-shortcuts -r -R -p / # This will reset keyboard settings to factory.)
2. Open settings-editor - you will see there is only commands/default and not commands/custom.
3. Open xfce4-keyboard-settings. It will immediately create commands/custom by copying the default settings.

Now compare the two Alt-F2 commands. The default one has startup-notification enabled, but the custom one (actually used) does not.

Now:

1. In xfce4-keyboard-settings, enable startup-notification on the Alt-F2 command. 2. Verify that the setting is set in settings editor. 
3. Now click "restore defaults" in tke keyboard settings. 

The startup-notification will be cleared again.
Comment 1 Alistair Buxton 2014-01-13 20:53:05 CET
This is seems to be a bug in libxfce4ui
Comment 2 Alistair Buxton 2014-01-13 21:01:15 CET
Created attachment 5314 
patch

This fixes the bug by relaxing the requirement that every shortcut property is type string.
Comment 3 Alistair Buxton 2014-01-30 00:14:53 CET
Created attachment 5339 
refresh patch, remove debug printf
Comment 4 Jannis Pohlmann editbugs 2014-07-25 15:41:28 CEST
Fix pushed to master:

commit 75b776202b4f21d9d87c4e87a07b6e9cb567f1f8
Author: Alistair Buxton <a.j.buxton@gmail.com>
Date:   Mon Jan 13 19:59:27 2014 +0000

    Don't drop startup notify hint when resetting to defaults (bug #10630)

Bug #10630

Reported by:
Alistair Buxton
Reported on: 2014-01-13
Last modified on: 2014-07-25

People

Assignee:
Jérôme Guelfucci
CC List:
4 users

Version

Attachments

patch (1.59 KB, patch)
2014-01-13 21:01 CET , Alistair Buxton
no flags
refresh patch, remove debug printf (1.49 KB, patch)
2014-01-30 00:14 CET , Alistair Buxton
no flags

Additional information