[Sugar-devel] [PATCH sugar] Fix shortcut for view source
simon at schampijer.de
Wed Apr 18 17:34:29 EDT 2012
On 04/16/2012 04:02 PM, Sascha Silbe wrote:
> Excerpts from Simon Schampijer's message of 2012-04-16 10:42:39 +0200:
>> -BOUND_KEYS = ['0xEC', '<alt><shift>v']
>> +BOUND_KEYS = ['XF86KbdLightOnOff', '<alt><shift>v']
> Is "Keyboard light on/off" really a key that we want to trigger View
> Source? Maybe it's what current OLPC images call the key, but that's
> If upstream (Linux) really is averse to adding a new definition for the
> "gear" key, the following existing one  sounds like a better fit:
> KEY_PROPS ("Display the properties of the current document" )
> That key would be better mapped to opening the Journal details view (for
> the current activity session), but if there's really push-back from the
> input subsystem maintainers, it would be a reasonable compromise.
>  https://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=blob_plain;f=include/linux/input.h;hb=HEAD
>  https://www.usb.org/developers/devclass_docs/Hut1_12v2.pdf (section 15.16)
I guess 'XF86WebCam' for 'open_search' or 'XF86MenuKB' for the 'frame'
is not much better . Not saying that those are good names but if you
want to have proper names you should change them all, but...
This is only a temporary solution. See  and  for using
gtk_accelerator_parse() or its more flexible variant
gtk_accelerator_parse_with_keycode() when we port the shell.
Hope that makes it clearer,
More information about the Sugar-devel