summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Thurman <tthurman@gnome.org>2008-10-12 15:09:48 +0000
committerThomas James Alexander Thurman <tthurman@src.gnome.org>2008-10-12 15:09:48 +0000
commitacd81439eb9b2140078408868873cfd1984c1649 (patch)
treed8186866dee84a969f8437571185a7fe284b06fc
parentb625ed254db23ff0f198e0cc325b747bb3b10039 (diff)
downloadmutter-acd81439eb9b2140078408868873cfd1984c1649.tar.gz
Fix annoying bug where alt-tab and friends would jump backwards a space on
2008-10-12 Thomas Thurman <tthurman@gnome.org> Fix annoying bug where alt-tab and friends would jump backwards a space on initial movement. * src/core/screen-bindings.h: although reversed bindings are necessarily reversible, don't set both bits in the constant, or when we test for them we'll get confused. svn path=/trunk/; revision=3950
-rw-r--r--ChangeLog9
-rw-r--r--src/core/screen-bindings.h19
2 files changed, 21 insertions, 7 deletions
diff --git a/ChangeLog b/ChangeLog
index 1079cc24a..4f6411c37 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,14 @@
2008-10-12 Thomas Thurman <tthurman@gnome.org>
+ Fix annoying bug where alt-tab and friends would jump
+ backwards a space on initial movement.
+
+ * src/core/screen-bindings.h: although reversed bindings
+ are necessarily reversible, don't set both bits in the
+ constant, or when we test for them we'll get confused.
+
+2008-10-12 Thomas Thurman <tthurman@gnome.org>
+
An attempt to make life a little easier for our beloved translators;
this has the same behaviour as before, but removes over thirty
translation strings.
diff --git a/src/core/screen-bindings.h b/src/core/screen-bindings.h
index 054d5caca..9823a5b30 100644
--- a/src/core/screen-bindings.h
+++ b/src/core/screen-bindings.h
@@ -81,7 +81,7 @@
#define BINDING_PER_WINDOW 0x01
#define BINDING_REVERSES 0x02
-#define BINDING_IS_REVERSED 0x06 /* REVERSES is implied */
+#define BINDING_IS_REVERSED 0x04
/* FIXME: There is somewhere better for these; remove them */
#define PANEL_MAIN_MENU -1
@@ -91,6 +91,9 @@
/***********************************/
+/* convenience, since in this file they must always be set together */
+#define REVERSES_AND_REVERSED (BINDING_REVERSES | BINDING_IS_REVERSED)
+
item (switch_to_workspace, "_1", 1, 0,
_("Switch to workspace 1"),
_("The keybinding that switches to workspace 1."),
@@ -185,7 +188,7 @@ item (switch, "_group", META_TAB_LIST_GROUP, BINDING_REVERSES,
_("Move between windows of an application with popup"),
_("The keybinding used to move focus between windows of an"
"application, using a popup window."), NULL)
-item (switch, "_group_backwards", META_TAB_LIST_GROUP, BINDING_IS_REVERSED,
+item (switch, "_group_backwards", META_TAB_LIST_GROUP, REVERSES_AND_REVERSED,
_("Move backwards between windows of an application with popup"),
_("The keybinding used to move focus backwards between windows"
"of an application, using a popup window."), NULL)
@@ -193,7 +196,7 @@ item (switch, "_windows", META_TAB_LIST_NORMAL, BINDING_REVERSES,
_("Move between windows with popup"),
_("The keybinding used to move focus between windows, "
"using a popup window."), "<Alt>Tab")
-item (switch, "_windows_backwards",META_TAB_LIST_NORMAL, BINDING_IS_REVERSED,
+item (switch, "_windows_backwards",META_TAB_LIST_NORMAL, REVERSES_AND_REVERSED,
_("Move focus backwards between windows using popup display"),
_("The keybinding used to move focus backwards between windows, "
"using a popup window."), NULL)
@@ -201,7 +204,7 @@ item (switch, "_panels", META_TAB_LIST_DOCKS, BINDING_REVERSES,
_("Move between panels and the desktop with popup"),
_("The keybinding used to move focus between panels and the desktop, "
"using a popup window."), "<Control><Alt>Tab")
-item (switch, "_panels_backwards", META_TAB_LIST_DOCKS, BINDING_IS_REVERSED,
+item (switch, "_panels_backwards", META_TAB_LIST_DOCKS, REVERSES_AND_REVERSED,
_("Move backwards between panels and the desktop with popup"),
_("The keybinding used to move focus backwards between panels "
"and the desktop, using a popup window."), NULL)
@@ -209,7 +212,7 @@ item (cycle, "_group", META_TAB_LIST_GROUP, BINDING_REVERSES,
_("Move between windows of an application immediately"),
_("The keybinding used to move focus between windows of an "
"application without a popup window."), "<Alt>F6")
-item (cycle, "_group_backwards", META_TAB_LIST_GROUP, BINDING_IS_REVERSED,
+item (cycle, "_group_backwards", META_TAB_LIST_GROUP, REVERSES_AND_REVERSED,
_("Move backwards between windows of an application immediately"),
_("The keybinding used to move focus backwards between windows "
"of an application without a popup window."), NULL)
@@ -217,7 +220,7 @@ item (cycle, "_windows", META_TAB_LIST_NORMAL, BINDING_REVERSES,
_("Move between windows immediately"),
_("The keybinding used to move focus between windows without "
"a popup window."), "<Alt>Escape")
-item (cycle, "_windows_backwards",META_TAB_LIST_NORMAL, BINDING_IS_REVERSED,
+item (cycle, "_windows_backwards",META_TAB_LIST_NORMAL, REVERSES_AND_REVERSED,
_("Move backwards between windows immediately"),
_("The keybinding used to move focus backwards between windows "
"without a popup window."), NULL)
@@ -225,7 +228,7 @@ item (cycle, "_panels", META_TAB_LIST_DOCKS, BINDING_REVERSES,
_("Move between panels and the desktop immediately"),
_("The keybinding used to move focus between panels and "
"the desktop, without a popup window."), "<Control><Alt>Escape")
-item (cycle, "_panels_backwards", META_TAB_LIST_DOCKS, BINDING_IS_REVERSED,
+item (cycle, "_panels_backwards", META_TAB_LIST_DOCKS, REVERSES_AND_REVERSED,
_("Move backward between panels and the desktop immediately"),
_("The keybinding used to move focus backwards between panels and "
"the desktop, without a popup window."), NULL)
@@ -303,5 +306,7 @@ item (run_terminal, "", 0, 0,
/* No descriptions because this is undocumented */
item (set_spew_mark, "", 0, 0, NULL, NULL, NULL)
+#undef REVERSES_AND_REVERSED
+
/* eof screen-bindings.h */