[sugar] [PATCH] remove non-contentious keyboard shortcuts

Martin Dengler martin at martindengler.com
Sat Apr 26 11:10:08 EDT 2008


Interpret Eben's reply to bemasc on #4646:

> > My primary concern is that activities be able to override and
> > deactivate shortcuts, for whatever bizarre uses they desire. Only
> > a handful of special keys should be trapped by Sugar, to ensure
> > minimal functionality like the ability to exit the activity.
>
> This is fair enough. The ability to override the defaults could be a
> reasonable option.

...as:

1) NOT allowing removal of any keyboard shortcut involving a keyboard
key that has an XO-specific image/glyph (this reserves all the F-keys,
and the XO-only keys on the keyboard and monitor like search, overlay,
rotate, dpad, etc.) and/or are obviously required for minimal sugar
functionality (this reserves ctrl-esc for exit/quit activity and
alt-tab and ctrl-alt-tab for obvious window-switching features) - this
is all justified, IMHO, by eben agreeing with bemacs's
"only a handful of special keys should be trapped by sugar, to ensure
minimal functionality . . ."; and

2) allowing removal of any keyboard shortcut that is not required by
minimal sugar functionality - this is all justified both as the
converse of what eben's agreement with bemasc means and also as a
consequence of eben explictly not objecting to removing the Ctrl-O
shortcut ("I wouldn't be sad if [the ctrl-o/open] shortcut went away").

The only contentiousness I can possibly see (if I've interpreted
eben/bemasc sympathetically) with this patch is that non-XO users of
sugar could lose access to the functionality now only available with
the XO-only keys (e.g., rotate, overlay).  I argue 1) this is not a
concern for any deployment; and 2) this can easily be addressed by
choosing different, less likely-to-interfere shortcuts.  As I'm less
comfortable choosing these shortcuts and without these shortcuts all
deployments can still make full use of Sugar/XO, I will defer any
choice of these alternates to a different patch so this patch can
easily be cherry-picked.
---
 src/view/keyhandler.py |    7 -------
 1 files changed, 0 insertions(+), 7 deletions(-)

diff --git a/src/view/keyhandler.py b/src/view/keyhandler.py
index 38f8a22..132eb7c 100644
--- a/src/view/keyhandler.py
+++ b/src/view/keyhandler.py
@@ -47,19 +47,12 @@ _actions_table = {
     '<ctrl>F11'      : 'volume_min',
     '<ctrl>F12'      : 'volume_max',
     '<alt>1'         : 'screenshot',
-    '<alt>f'         : 'frame',
     '0x93'           : 'frame',
     '0xEB'           : 'rotate',
-    '<alt>r'         : 'rotate',
-    '<alt>q'         : 'quit_emulator',
     '<alt>Tab'       : 'next_window',
-    '<alt>n'         : 'next_window',
     '<ctrl><alt>Tab' : 'previous_window',
-    '<alt>p'         : 'previous_window',
     '<ctrl>Escape'   : 'close_window',
-    '<ctrl>q'        : 'close_window',
     '0xDC'           : 'open_search',
-    '<alt>s'         : 'say_text'
 }
 
 J_DBUS_SERVICE = 'org.laptop.Journal'
-- 
1.5.4.1



More information about the Sugar mailing list