keybinding properties doesn't remove a shortcut correctly

Bug #515670 reported by Sakari Vaelma
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
GNOME Settings Daemon
Fix Released
Medium
gnome-settings-daemon (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: gnome-control-center

On Ubuntu 9.10 (localized for Finnish) gnome-keybinding-properties doesn't remove a shortcut correctly when backspace is pressed. The previous assignment disappears, but the functionality stays. In some cases the original function of a button (try that with Delete for example) disappears and nothing else happens. It seems that gnome-keybinding-properties changes the key value in gconf to an empty string while it should be "disabled".

Steps to reproduce:
1) Open gnome-keybinding-properties
2) Click on "Hide window" (or whatever the equivalent in English would be)
3) Press Delete to assign it as the key binding
4) Test that it works as wanted (it does for me)
5) Click on "Hide window" again
6) Press Backspace to remove the binding
7) See the binding disappear in the list
8) Now press Delete to notice random behaviour (hides the window, delete button just not working for deleting text etc.)
9) Check out gconf-editor to see what the value in /apps/metacity/window_keybindings for the key "lower" would be. That seems to be now set to an empty string while it should be "disabled".
10) If you fix it in gconf-editor, everything works as supposed.

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your bug report. The issue is an upstream one and it would be nice if somebody having it could send the bug the to the people writting the software (https://wiki.ubuntu.com/Bugs/Upstream/GNOME)

Changed in gnome-control-center (Ubuntu):
importance: Undecided → Low
Revision history for this message
Steven Flintham (saf) wrote :

I have experienced what I believe to be the same bug in Ubuntu 10.04. Since I couldn't see any sign of the above bug having been forwarded, I have raised the following in the GNOME bug tracker: https://bugzilla.gnome.org/show_bug.cgi?id=630411

Revision history for this message
Steven Flintham (saf) wrote :

I accidentally filed a duplicate with GNOME. The 'primary' upstream bug is https://bugzilla.gnome.org/show_bug.cgi?id=625228.

affects: gnome-control-center (Ubuntu) → gnome-settings-daemon (Ubuntu)
Changed in gnome-settings-daemon (Ubuntu):
status: New → Triaged
Changed in gnome-settings-daemon:
importance: Unknown → Medium
status: Unknown → New
Changed in gnome-settings-daemon:
status: New → Fix Released
Changed in gnome-settings-daemon (Ubuntu):
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gnome-settings-daemon - 3.3.5-0ubuntu1

---------------
gnome-settings-daemon (3.3.5-0ubuntu1) precise; urgency=low

  * New upstream version (lp: #931656), refreshed patches,
    thanks Jeremy Bicha for starting the work on the new version:
    - change the sound level by device (lp: #340720)
    - correctly unassign keybindings (lp: #515670)
    - define keybinding to take a screenshot of a selection (lp: #625518)
    - resolve some segfaults (lp: #865286)
  * debian/control.in:
    - build-depends on libgudev-1.0-dev
    - breaks unity-greeter (<< 0.2.1-0ubuntu1) due to the wacom schema rename
    - build-depends on libwacom
    - updated gtk, gnome-desktop3, g-d-s, upower requirements
  * debian/gnome-settings-daemon.gsettings-override:
    - don't set deprecated "sleep-inactive-battery" key
  * debian/patches/90_set_gmenus_xsettings.patch:
    - refreshed, we don't handle gnome-shell xsettings yet because we still
      ship 3.2, that patch will need an update if we go for the new version
  * debian/patches/git_gsettings_write_only_when_required.patch;
    debian/patches/50_add_dell_backlight.patch:
    - dropped the fix is in the new version
  * debian/patches/47_delay_pa_connect_to_idle.patch:
    - commented, will need to be updated if still needed
  * debian/patches/revert_git_datetime_dropping.patch:
    - revert the datetime service dropping, ubuntu-system-service doesn't
      implement the timedated interfaces
  * debian/patches/revert_git_dropping_gconf_wrapper.patch:
    - revert the gconf code dropping, we might still need the settings
      gateway for the lts (though it's a bit hackish)
  * debian/patches/revert_git_use_gsetting_keybindings.patch:
    - revert the keybindings switch to gsettings, compiz is not ready,
      it's likely possible to update the patch to do either according to
      the session in use at runtime if somebody wants to work on that
  * debian/patches/revert_git_a11y_gsettings.patch:
    - similar to the previous one
  * debian/patches/revert_git_stop_using_gconf.patch:
    - don't stop using gconf, the other patches listed before need it
  * debian/rules:
    - don't use -Bsymbolic-functions to workaround upstream issues
    - drop deprecated --enable-pulse option
    - use --disable-systemd
  * debian/watch:
    - watch for .xz
 -- Sebastien Bacher <email address hidden> Wed, 15 Feb 2012 11:41:52 +0100

Changed in gnome-settings-daemon (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.