From b0cfcfaf3033bd4b928acb28bd893c7d87336f93 Mon Sep 17 00:00:00 2001
From: Yannick Tivisse <yti@odoo.com>
Date: Tue, 26 Apr 2016 13:37:27 +0200
Subject: [PATCH] [IMP] web: Shortcuts easier to use (lowercase)

By putting the shortcuts lowercase, it is easier to remember and to user.

Instead of Alt + Shift + letter, we have Alt + letter for Windows and Linus users
Instead of Control + Alt + Shift + letter, we have Control + Alt + letter for mac users

The combinations Alt + d and Alt + e are reserved to the browsers, so other letters are chosen
for these actions
---
 addons/web/static/src/js/views/list_view.js    |  2 +-
 addons/web/static/src/xml/base.xml             | 18 +++++++++---------
 addons/web_kanban/static/src/js/kanban_view.js |  2 +-
 .../web_kanban/static/src/xml/web_kanban.xml   |  2 +-
 4 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/addons/web/static/src/js/views/list_view.js b/addons/web/static/src/js/views/list_view.js
index 97b0a5a3879..ab71804b33b 100644
--- a/addons/web/static/src/js/views/list_view.js
+++ b/addons/web/static/src/js/views/list_view.js
@@ -35,7 +35,7 @@ var row_decoration = [
 
 var ListView = View.extend({
     _template: 'ListView',
-    accesskey: "L",
+    accesskey: "l",
     defaults: _.extend({}, View.prototype.defaults, {
         // records can be selected one by one
         selectable: true,
diff --git a/addons/web/static/src/xml/base.xml b/addons/web/static/src/xml/base.xml
index 2cb71a0574b..5fe0d350d0f 100644
--- a/addons/web/static/src/xml/base.xml
+++ b/addons/web/static/src/xml/base.xml
@@ -358,14 +358,14 @@
     <t t-if="!widget.no_leaf and widget.options.action_buttons !== false">
         <div class="o_list_buttons">
             <t t-if="widget.options.addable and widget.is_action_enabled('create')">
-                <button type="button" class="btn btn-primary btn-sm o_list_button_add" accesskey="C">
+                <button type="button" class="btn btn-primary btn-sm o_list_button_add" accesskey="c">
                     <t t-esc="widget.options.addable"/>
                 </button>
             </t>
-            <button type="button" class="btn btn-primary btn-sm o_list_button_save" accesskey="S">
+            <button type="button" class="btn btn-primary btn-sm o_list_button_save" accesskey="s">
                 Save
             </button>
-            <button type="button" class="btn btn-default btn-sm o_list_button_discard" accesskey="D">
+            <button type="button" class="btn btn-default btn-sm o_list_button_discard" accesskey="j">
                 Discard
             </button>
         </div>
@@ -430,21 +430,21 @@
 <t t-name="FormView.buttons">
     <div class="o_form_buttons_view">
         <button t-if="widget.is_action_enabled('edit')" type="button"
-                class="btn btn-primary btn-sm o_form_button_edit" accesskey="E">
+                class="btn btn-primary btn-sm o_form_button_edit" accesskey="a">
             Edit
         </button>
         <button t-if="widget.is_action_enabled('create')" type="button"
-                class="btn btn-default btn-sm o_form_button_create" accesskey="C">
+                class="btn btn-default btn-sm o_form_button_create" accesskey="c">
             Create
         </button>
     </div>
     <div class="o_form_buttons_edit">
         <button type="button"
-                class="btn btn-primary btn-sm o_form_button_save" accesskey="S">
+                class="btn btn-primary btn-sm o_form_button_save" accesskey="s">
             Save
         </button>
         <button type="button"
-                class="btn btn-default btn-sm o_form_button_cancel" accesskey="D">
+                class="btn btn-default btn-sm o_form_button_cancel" accesskey="j">
             Discard
         </button>
     </div>
@@ -1233,8 +1233,8 @@
     <span class="btn-group btn-group-sm">
         <!-- accesskeys not wanted in X2Many widgets -->
         <t t-if="!widget.getParent().x2m">
-            <t t-set="att_prev" t-value="{'accesskey': 'O'}" />
-            <t t-set="att_next" t-value="{'accesskey': 'P'}" />
+            <t t-set="att_prev" t-value="{'accesskey': 'p'}" />
+            <t t-set="att_next" t-value="{'accesskey': 'n'}" />
         </t>
         <button type="button" class="fa fa-chevron-left btn btn-icon o_pager_previous" t-att="att_prev"/>
         <button type="button" class="fa fa-chevron-right btn btn-icon o_pager_next" t-att="att_next"/>
diff --git a/addons/web_kanban/static/src/js/kanban_view.js b/addons/web_kanban/static/src/js/kanban_view.js
index 6f225f23062..bd5fd4563f0 100644
--- a/addons/web_kanban/static/src/js/kanban_view.js
+++ b/addons/web_kanban/static/src/js/kanban_view.js
@@ -25,7 +25,7 @@ var ColumnQuickCreate = quick_create.ColumnQuickCreate;
 var fields_registry = kanban_widgets.registry;
 
 var KanbanView = View.extend({
-    accesskey: "K",
+    accesskey: "k",
     className: "o_kanban_view",
     custom_events: {
         'kanban_record_open': 'open_record',
diff --git a/addons/web_kanban/static/src/xml/web_kanban.xml b/addons/web_kanban/static/src/xml/web_kanban.xml
index 01593bf0a37..e3ca582bf3e 100644
--- a/addons/web_kanban/static/src/xml/web_kanban.xml
+++ b/addons/web_kanban/static/src/xml/web_kanban.xml
@@ -2,7 +2,7 @@
 
 <t t-name="KanbanView.buttons">
     <div>
-        <button type="button" class="btn btn-primary btn-sm o-kanban-button-new" accesskey="C">
+        <button type="button" class="btn btn-primary btn-sm o-kanban-button-new" accesskey="c">
             <t t-esc="widget.options.create_text || _t('Create')"/>
         </button>
     </div>