[PATCH] fdo#32368 - Link both Ctrl+Y as Ctrl+Shift+Z as shortcut key...
Joren De Cuyper (via Code Review)
gerrit at gerrit.libreoffice.org
Sat May 11 03:22:26 PDT 2013
Hi,
I have submitted a patch for review:
https://gerrit.libreoffice.org/3853
To pull it, you can do:
git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/53/3853/1
fdo#32368 - Link both Ctrl+Y as Ctrl+Shift+Z as shortcut keys for Redo
As discussed on the UX-advice and bug report itself,
we agreed to link both shortcut keys to the redo command.
Due http://opengrok.libreoffice.org/xref/core/framework/source/inc/accelerators/acceleratorcache.hxx#75 :
/*map commands to keys in relation 1:n. First key is interpreted as preferred one!*/ the shortcut key
that is mentioned in the menu entry is ctrl+y (or cmd+y for Mac users).
UX-advice thread: http://lists.freedesktop.org/archives/libreoffice-ux-advise/2013-May/002033.html
Change-Id: I8d766b300d79e9ebd85dc5ae7bb327cf9c1a7c41
---
M officecfg/registry/data/org/openoffice/Office/Accelerators.xcu
1 file changed, 86 insertions(+), 80 deletions(-)
diff --git a/officecfg/registry/data/org/openoffice/Office/Accelerators.xcu b/officecfg/registry/data/org/openoffice/Office/Accelerators.xcu
index d2d7069..2ef7c67 100644
--- a/officecfg/registry/data/org/openoffice/Office/Accelerators.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/Accelerators.xcu
@@ -190,7 +190,13 @@
<value xml:lang="en-US">.uno:Repeat</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace" install:module="unxwnt">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
+ <prop oor:name="Command">
+ <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
+ <value xml:lang="en-US">.uno:Redo</value>
+ </prop>
+ </node>
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
@@ -200,12 +206,6 @@
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Undo</value>
- </prop>
- </node>
- <node oor:name="Z_SHIFT_MOD1" oor:op="replace" install:module="macosx">
- <prop oor:name="Command">
- <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
- <value xml:lang="en-US">.uno:Redo</value>
</prop>
</node>
</node>
@@ -1132,7 +1132,13 @@
<value xml:lang="en-US">.uno:PasteSpecial</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace" install:module="unxwnt">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
+ <prop oor:name="Command">
+ <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
+ <value xml:lang="en-US">.uno:Redo</value>
+ </prop>
+ </node>
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
@@ -1142,12 +1148,6 @@
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Undo</value>
- </prop>
- </node>
- <node oor:name="Z_SHIFT_MOD1" oor:op="replace" install:module="macosx">
- <prop oor:name="Command">
- <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
- <value xml:lang="en-US">.uno:Redo</value>
</prop>
</node>
<node oor:name="C_MOD1_MOD2" oor:op="replace">
@@ -1248,7 +1248,13 @@
<value xml:lang="en-US">.uno:Cut</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace" install:module="unxwnt">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
+ <prop oor:name="Command">
+ <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
+ <value xml:lang="en-US">.uno:Redo</value>
+ </prop>
+ </node>
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
@@ -1258,12 +1264,6 @@
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Undo</value>
- </prop>
- </node>
- <node oor:name="Z_SHIFT_MOD1" oor:op="replace" install:module="macosx">
- <prop oor:name="Command">
- <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
- <value xml:lang="en-US">.uno:Redo</value>
</prop>
</node>
</node>
@@ -1668,7 +1668,13 @@
<value xml:lang="en-US">.uno:Cut</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace" install:module="unxwnt">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
+ <prop oor:name="Command">
+ <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
+ <value xml:lang="en-US">.uno:Redo</value>
+ </prop>
+ </node>
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
@@ -1678,12 +1684,6 @@
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Undo</value>
- </prop>
- </node>
- <node oor:name="Z_SHIFT_MOD1" oor:op="replace" install:module="macosx">
- <prop oor:name="Command">
- <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
- <value xml:lang="en-US">.uno:Redo</value>
</prop>
</node>
<node oor:name="BRACKETLEFT_MOD1" oor:op="replace">
@@ -2447,7 +2447,13 @@
<value xml:lang="en-US">.uno:RemoveDirectCharFormats</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace" install:module="unxwnt">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
+ <prop oor:name="Command">
+ <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
+ <value xml:lang="en-US">.uno:Redo</value>
+ </prop>
+ </node>
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
@@ -2457,12 +2463,6 @@
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Undo</value>
- </prop>
- </node>
- <node oor:name="Z_SHIFT_MOD1" oor:op="replace" install:module="macosx">
- <prop oor:name="Command">
- <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
- <value xml:lang="en-US">.uno:Redo</value>
</prop>
</node>
</node>
@@ -2927,7 +2927,13 @@
<value xml:lang="en-US">.uno:Cut</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace" install:module="unxwnt">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
+ <prop oor:name="Command">
+ <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
+ <value xml:lang="en-US">.uno:Redo</value>
+ </prop>
+ </node>
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
@@ -2937,12 +2943,6 @@
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Undo</value>
- </prop>
- </node>
- <node oor:name="Z_SHIFT_MOD1" oor:op="replace" install:module="macosx">
- <prop oor:name="Command">
- <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
- <value xml:lang="en-US">.uno:Redo</value>
</prop>
</node>
<node oor:name="BRACKETLEFT_MOD1" oor:op="replace">
@@ -3001,7 +3001,13 @@
<value xml:lang="en-US">.uno:FullScreen</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace" install:module="unxwnt">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
+ <prop oor:name="Command">
+ <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
+ <value xml:lang="en-US">.uno:Redo</value>
+ </prop>
+ </node>
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
@@ -3011,12 +3017,6 @@
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Undo</value>
- </prop>
- </node>
- <node oor:name="Z_SHIFT_MOD1" oor:op="replace" install:module="macosx">
- <prop oor:name="Command">
- <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
- <value xml:lang="en-US">.uno:Redo</value>
</prop>
</node>
</node>
@@ -3774,7 +3774,13 @@
<value xml:lang="en-US">.uno:RemoveDirectCharFormats</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace" install:module="unxwnt">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
+ <prop oor:name="Command">
+ <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
+ <value xml:lang="en-US">.uno:Redo</value>
+ </prop>
+ </node>
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
@@ -3784,12 +3790,6 @@
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Undo</value>
- </prop>
- </node>
- <node oor:name="Z_SHIFT_MOD1" oor:op="replace" install:module="macosx">
- <prop oor:name="Command">
- <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
- <value xml:lang="en-US">.uno:Redo</value>
</prop>
</node>
</node>
@@ -4553,13 +4553,13 @@
<value xml:lang="en-US">.uno:RemoveDirectCharFormats</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace" install:module="unxwnt">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
</prop>
</node>
- <node oor:name="Z_SHIFT_MOD1" oor:op="replace" install:module="macosx">
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
@@ -5290,7 +5290,13 @@
<value xml:lang="en-US">.uno:RemoveDirectCharFormats</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace" install:module="unxwnt">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
+ <prop oor:name="Command">
+ <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
+ <value xml:lang="en-US">.uno:Redo</value>
+ </prop>
+ </node>
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
@@ -5300,12 +5306,6 @@
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Undo</value>
- </prop>
- </node>
- <node oor:name="Z_SHIFT_MOD1" oor:op="replace" install:module="macosx">
- <prop oor:name="Command">
- <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
- <value xml:lang="en-US">.uno:Redo</value>
</prop>
</node>
</node>
@@ -6069,7 +6069,13 @@
<value xml:lang="en-US">.uno:RemoveDirectCharFormats</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace" install:module="unxwnt">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
+ <prop oor:name="Command">
+ <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
+ <value xml:lang="en-US">.uno:Redo</value>
+ </prop>
+ </node>
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
@@ -6079,12 +6085,6 @@
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Undo</value>
- </prop>
- </node>
- <node oor:name="Z_SHIFT_MOD1" oor:op="replace" install:module="macosx">
- <prop oor:name="Command">
- <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
- <value xml:lang="en-US">.uno:Redo</value>
</prop>
</node>
</node>
@@ -6836,7 +6836,13 @@
<value xml:lang="en-US">.uno:RemoveDirectCharFormats</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace" install:module="unxwnt">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
+ <prop oor:name="Command">
+ <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
+ <value xml:lang="en-US">.uno:Redo</value>
+ </prop>
+ </node>
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
@@ -6846,12 +6852,6 @@
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Undo</value>
- </prop>
- </node>
- <node oor:name="Z_SHIFT_MOD1" oor:op="replace" install:module="macosx">
- <prop oor:name="Command">
- <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
- <value xml:lang="en-US">.uno:Redo</value>
</prop>
</node>
</node>
@@ -7448,7 +7448,13 @@
<value xml:lang="en-US">.uno:SelectAllEdits</value>
</prop>
</node>
- <node oor:name="Y_MOD1" oor:op="replace">
+ <node oor:name="Z_SHIFT_MOD1" oor:op="replace">
+ <prop oor:name="Command">
+ <value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
+ <value xml:lang="en-US">.uno:Redo</value>
+ </prop>
+ </node>
+ <node oor:name="Y_MOD1" oor:op="fuse">
<prop oor:name="Command">
<value xml:lang="x-no-translate">I10N SHORTCUTS - NO TRANSLATE</value>
<value xml:lang="en-US">.uno:Redo</value>
--
To view, visit https://gerrit.libreoffice.org/3853
To unsubscribe, visit https://gerrit.libreoffice.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8d766b300d79e9ebd85dc5ae7bb327cf9c1a7c41
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Joren De Cuyper <joren.libreoffice at telenet.be>
More information about the LibreOffice
mailing list