[gnome-love] Re: ten maybe-possibly-fairly easy-fix bugs for new hacking volunteers

[Bill Haneman <bill haneman sun com>]

On Sat, 2002-04-20 at 06:37, R.I.P. Deaddog wrote:

> > Without further ado:
> >
> > 1) two commands use same keyboard shortcut:
> > http://bugzilla.gnome.org/show_bug.cgi?id=75248
> [......]
>
> Item 1 done. Luckily this one doesn't require C skill at all *grin*
> Patched attached, and it fixes another duplicate shortcut key
> in Accessibility keyboard tab too. OK to commit?
>
> Abel

I don't believe this is necessarily the right fix; please note
jacob berkman's comments from April 19.  He reports that the
correct behavior is currently available; i.e. pressing a
shortcut for which there are two actions cycles between the
actions.

Thanks for not changing any accessibility keybindings without
getting the go-ahead from the accessibility team;

in fact I think we should get input from Calum, Seth, etc.
(i.e. usability and keynav folks) before making any keybindings 
changes, even if they appear to be "fixes"

thanks

Bill
> > Luis
>
> -- 
> Abel Cheung
> GPG Key: (0xC67186FF) http://deaddog.org/gpg.asc
> ----

> Index: capplets/keyboard/gnome-keyboard-properties.glade
> ===================================================================
> RCS file: /cvs/gnome/gnome-control-center/capplets/keyboard/gnome-keyboard-properties.glade,v
> retrieving revision 1.7
> diff -u -r1.7 gnome-keyboard-properties.glade
> --- capplets/keyboard/gnome-keyboard-properties.glade 18 Apr 2002 18:03:46 -0000      1.7
> +++ capplets/keyboard/gnome-keyboard-properties.glade 20 Apr 2002 05:31:38 -0000
> @@ -361,7 +361,7 @@
>                               <widget class="GtkCheckButton" id="cursor_toggle">
>                                 <property name="visible">True</property>
>                                 <property name="can_focus">True</property>
> -                               <property name="label" translatable="yes">_Cursor blinks in text 
> fields</property>
> +                               <property name="label" translatable="yes">C_ursor blinks in text 
> fields</property>
>                                 <property name="use_underline">True</property>
>                                 <property name="relief">GTK_RELIEF_NORMAL</property>
>                                 <property name="active">False</property>
> Index: capplets/accessibility/keyboard/gnome-accessibility-keyboard-properties.glade2
> ===================================================================
> RCS file: 
> /cvs/gnome/gnome-control-center/capplets/accessibility/keyboard/gnome-accessibility-keyboard-properties.glade2,v
> retrieving revision 1.5
> diff -u -r1.5 gnome-accessibility-keyboard-properties.glade2
> --- capplets/accessibility/keyboard/gnome-accessibility-keyboard-properties.glade2    18 Apr 2002 16:09:14 
> -0000      1.5
> +++ capplets/accessibility/keyboard/gnome-accessibility-keyboard-properties.glade2    20 Apr 2002 05:31:39 
> -0000
> @@ -512,7 +512,7 @@
>                         <property name="visible">True</property>
>                         <property name="tooltip" translatable="yes">Perform multiple simultaneous key press 
> operations by pressing modifier keys in sequence.</property>
>                         <property name="can_focus">True</property>
> -                       <property name="label" translatable="yes">Enable _Sticky Keys</property>
> +                       <property name="label" translatable="yes">Enable St_icky Keys</property>
>                         <property name="use_underline">True</property>
>                         <property name="relief">GTK_RELIEF_NORMAL</property>
>                         <property name="active">True</property>