From 0a463c5c40a4d4724610b19e35b00a33d6eb83cf Mon Sep 17 00:00:00 2001
From: Sylvain LE GAL <sylvain.legal@grap.coop>
Date: Fri, 18 Jun 2021 10:47:34 +0200
Subject: [PATCH] [IMP+FIX] web_responsive : fix shortcut display and add new
 shortcuts (u) for the User menu

---
 web_responsive/readme/USAGE.rst        |  5 ++-
 web_responsive/static/src/xml/menu.xml | 48 ++++++++++++++++++++++++++
 2 files changed, 52 insertions(+), 1 deletion(-)

diff --git a/web_responsive/readme/USAGE.rst b/web_responsive/readme/USAGE.rst
index a8a1c12bd..d340c7872 100644
--- a/web_responsive/readme/USAGE.rst
+++ b/web_responsive/readme/USAGE.rst
@@ -1,6 +1,9 @@
 The following keyboard shortcuts are implemented:
 
-* Toggle app drawer - ``Alt + Shift + H``
+* Toggle app drawer - ``Alt + Shift + A``
 * Navigate app search results - Arrow keys
 * Choose app result - ``Enter``
 * ``Esc`` to close app drawer
+
+
+Also you can access to the user menu, with ``Alt + Shift + U``
diff --git a/web_responsive/static/src/xml/menu.xml b/web_responsive/static/src/xml/menu.xml
index 83c8a60a7..4a4b8ee68 100644
--- a/web_responsive/static/src/xml/menu.xml
+++ b/web_responsive/static/src/xml/menu.xml
@@ -1,10 +1,58 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <templates id="template" xml:space="preserve">
 
+<t t-extend="UserMenu">
+    <t t-jquery=".dropdown-toggle" t-operation="attributes">
+        <attribute name="accesskey">u</attribute>
+    </t>
+</t>
+
 <div t-extend="UserMenu.shortcuts">
+    <!-- Replace "a" shortcut by "e" (for 'Edit a Record') -->
+    <t t-jquery="table.o_shortcut_table > tbody > tr > td:nth-child(2) > span:nth-child(2).o_key:contains('a')" t-operation="replace">
+        <span class="o_key">e</span>
+    </t>
+    <t t-jquery="table.o_shortcut_table > tbody > tr > td:nth-child(3) > span:nth-child(3).o_key:contains('a')" t-operation="replace">
+        <span class="o_key">e</span>
+    </t>
+
+    <!-- Replace "j" shortcut by "d" (for 'Discard a record modification') -->
+    <t t-jquery="table.o_shortcut_table > tbody > tr > td:nth-child(2) > span:nth-child(2).o_key:contains('j')" t-operation="replace">
+        <span class="o_key">d</span>
+    </t>
+    <t t-jquery="table.o_shortcut_table > tbody > tr > td:nth-child(3) > span:nth-child(3).o_key:contains('j')" t-operation="replace">
+        <span class="o_key">d</span>
+    </t>
+
+    <!-- Add Shift for all windows / linux shortcuts -->
     <t t-jquery="table.o_shortcut_table > tbody > tr > td:nth-child(2) > span:first-child" t-operation="after">
         + <span class="o_key">Shift</span>
     </t>
+
+    <!-- Add new accesskeys -->
+    <t t-jquery=".o_shortcut_table tbody" t-operation="append">
+        <tr>
+            <td align="left">Select menu</td>
+            <td>
+                <span class="o_key">Alt</span> + <span class="o_key">Shift</span> + <span class="o_key">a</span>
+            </td>
+            <td>
+                <span class="o_key">Control</span> + <span class="o_key">Alt</span> + <span class="o_key">a</span>
+            </td>
+        </tr>
+    </t>
+
+    <t t-jquery=".o_shortcut_table tbody" t-operation="append">
+        <tr>
+            <td align="left">User menu</td>
+            <td>
+                <span class="o_key">Alt</span> + <span class="o_key">Shift</span> + <span class="o_key">u</span>
+            </td>
+            <td>
+                <span class="o_key">Control</span> + <span class="o_key">Alt</span> + <span class="o_key">u</span>
+            </td>
+        </tr>
+    </t>
 </div>
 
 </templates>