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@telenet.be>
Context
- [PATCH] fdo#32368 - Link both Ctrl+Y as Ctrl+Shift+Z as shortcut key... · Joren De Cuyper (via Code Review)
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.