! 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 !
'Restore numlock state on startup' not working.
Status:
RESOLVED: FIXED
Product:
Xfce4-settings
Component:
Keyboard Settings

Comments

Description Sergio 2019-06-26 19:19:09 CEST
xfce4-settings-4.13.6-1.fc30 (fc30 means Fedora 30)

Forum thread https://forum.xfce.org/viewtopic.php?id=13109

- There is an option in the settings editor (xfconf) to set numlock on or off by default but it isn't present in the keyboard settings GUI. So I went with setting numlock to on and checking to save state, but this isn't working.
Now I have what I need as by the forum thread I discovered the setting to turn it on as default but if, for instance, I disable it, it starts enabled because of 'restore state' not working.
Comment 1 Theo Linkspfeifer editbugs 2019-06-27 01:53:11 CEST
https://git.xfce.org/xfce/xfce4-settings/tree/xfsettingsd/keyboards.c#n306

It appears that xfconf_channel_set_bool() does not create the property in case it is missing.
Comment 2 Theo Linkspfeifer editbugs 2019-06-27 15:16:36 CEST
Created attachment 8694 
patch

Closing the D-Bus connection after (instead of before) shutting down xfconf fixes this bug. The property is now created or updated whenever xfsettingsd is terminated.

Also, the toggle button in the settings GUI was incorrectly set to 'active'.
Comment 3 Git Bot editbugs 2019-06-30 01:35:49 CEST
Theo Linkspfeifer referenced this bugreport in commit 5bfabccb9a13e4612b6017bdc9a407f606718580

Fix numlock state restoration (Bug #15642)

https://git.xfce.org/xfce/xfce4-settings/commit?id=5bfabccb9a13e4612b6017bdc9a407f606718580
Comment 4 Sean Davis editbugs 2019-06-30 01:36:25 CEST
Thanks for the patch, looks good to me. Committed above.

Bug #15642

Reported by:
Sergio
Reported on: 2019-06-26
Last modified on: 2019-06-30

People

Assignee:
Xfce Bug Triage
CC List:
2 users

Version

Version:
unspecified

Attachments

patch (1.95 KB, patch)
2019-06-27 15:16 CEST , Theo Linkspfeifer
no flags

Additional information