Date: prev next · Thread: first prev next last
2013 Archives by date, by thread · List index


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


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.