Merge lp:~openerp-dev/openerp-web/trunk-improve-shortcutkeys-hotkeys-bth into lp:openerp-web

[Harry (OpenERP)]

[lp:~openerp-dev/openerp-web/trunk-improve-shortcutkeys-hotkeys-bth]

3875. By Bhumi Thakkar (Open ERP) on 2013-12-06

[MERGE]With Trunk.

3876. By Dharmraj Zala(OpenERP) on 2014-01-30

[MERGE] with trunk

3877. By Dharmraj Zala(OpenERP) on 2014-02-03

[MERGE] with trunk

Unmerged revisions

3877. By Dharmraj Zala(OpenERP) on 2014-02-03

[MERGE] with trunk

3876. By Dharmraj Zala(OpenERP) on 2014-01-30

[MERGE] with trunk

3875. By Bhumi Thakkar (Open ERP) on 2013-12-06

[MERGE]With Trunk.

3874. By Bhumi Thakkar (Open ERP) on 2013-11-20

[MERGE]Merge trunk.

3873. By Bhumi Thakkar (Open ERP) on 2013-11-15

[IMP]Removed code shortcuts for switch view and links. added for pager on left & right arrows.

3872. By Bhumi Thakkar (Open ERP) on 2013-11-15

[MERGE]Merge trunk.

3871. By Bhumi Thakkar (Open ERP) on 2013-10-31

[MERGE]Merge trunk.

3870. By Bhumi Thakkar (Open ERP) on 2013-10-31

[IMP]IMproved code for accesskeys.Accesskeys in lowercase.

3869. By Bhumi Thakkar (Open ERP) on 2013-10-29

[MERGE]Merge trunk.

3868. By Bhumi Thakkar (Open ERP) on 2013-10-28

[IMP]Using hotkeys events called.

=== modified file 'addons/web/static/src/css/base.css'
--- addons/web/static/src/css/base.css	2014-01-31 00:52:02 +0000
+++ addons/web/static/src/css/base.css	2014-02-03 06:03:15 +0000
@@ -1,4 +1,4 @@
-@charset "UTF-8";
+@charset "utf-8";
 @font-face {
   font-family: "mnmliconsRegular";
   src: url("/web/static/src/font/mnmliconsv21-webfont.eot") format("eot");
@@ -893,7 +893,7 @@
   background-image: -moz-linear-gradient(top, #fc8787, maroon);
   background-image: -ms-linear-gradient(top, #fc8787, maroon);
   background-image: -o-linear-gradient(top, #fc8787, maroon);
-  background-image: linear-gradient(to bottom, #fc8787, #800000);
+  background-image: linear-gradient(to bottom, #fc8787, maroon);
 }
 .openerp .navbar .oe_topbar_anonymous_login a {
   display: block;
@@ -3153,6 +3153,10 @@
   margin: 3px 3px 0 !important;
 }
 
+.accesshighlight:first-letter, .highlight {
+  text-decoration: underline;
+}
+
 @-moz-document url-prefix() {
   .openerp .oe_searchview .oe_searchview_search {
     top: -1px;
@@ -3434,6 +3438,7 @@
     overflow: hidden !important;
   }
 }
+
 .ui-icon {
   width: 18px;
   height: 18px;

=== modified file 'addons/web/static/src/css/base.sass'
--- addons/web/static/src/css/base.sass	2014-01-31 00:52:02 +0000
+++ addons/web/static/src/css/base.sass	2014-02-03 06:03:15 +0000
@@ -2531,6 +2531,9 @@
     .oe_msg_subtype_check
         margin: 3px 3px 0 !important
     // }}}
+    
+.accesshighlight:first-letter, .highlight
+    text-decoration: underline
 
 @-moz-document url-prefix()
     .openerp

=== modified file 'addons/web/static/src/js/chrome.js'
--- addons/web/static/src/js/chrome.js	2014-01-16 18:15:19 +0000
+++ addons/web/static/src/js/chrome.js	2014-02-03 06:03:15 +0000
@@ -9,6 +9,93 @@
 var QWeb = instance.web.qweb,
     _t = instance.web._t;
 
+/* using jquery.hotkeys */
+$(document).bind('keydown', 'alt', function(e) {
+    highlightAccessKeys();
+}).bind('keydown', 'shift', function(e) {
+    highlightAccessKeys();
+}).keyup(function() {
+    unhighlightAccessKeys();
+});
+
+/* replace character with span tag into exist element */
+function replaceHtml(key, el) {
+    $(el).html($(el).text().replace(key, '<span class ="highlight">' + key + '</span>'))
+}
+/* 
+ * find accesskey is assigned to anyother element. if assigned then find nextone character else assign. 
+ * */
+function addkey(key, el, index, flag) {
+    if ($.inArray(key,this.keys) == -1) {
+        $(el).attr('accesskey', key.toLowerCase());
+        if (index == 0) {
+            replaceHtml($(el).attr('accesskey').toUpperCase(), el);
+        } else {
+            if (flag) keyval = key;
+            else keyval = key.toLowerCase();
+            replaceHtml(keyval, el);
+        }
+        this.keys.push(key);
+    } else {
+        index++;
+        val = $(el).text().replace(/\s/g, '').charAt(index);
+        conval = $(el).text().replace(/\s/g, '').charAt(index).toUpperCase();
+        key = $(el).text().replace(/\s/g, '').charAt(index).toUpperCase();
+        if(index < ($.trim($(el).text()).length + 1)){
+            addkey(key, el, index, val == conval);
+        }
+    }
+}
+/* 
+ * highlightAccesskeys() - find element which has accesskey attribute and also element which has to assign accesskey attribute
+ * */
+function highlightAccessKeys() {
+    var keys = [];
+
+    // accesskey for header
+    $hedaer = $('.oe_view_manager_current > table');
+
+    _.each($($hedaer).find('.oe_view_manager_buttons > div'), function(el) {
+        if ($(el).css('display') == 'block') {
+            _.each($(el).find('button[accesskey]'), function (btn) {
+                replaceHtml($(btn).attr('accesskey').toUpperCase(), btn);
+                keys.push($(btn).attr('accesskey').toUpperCase())
+            });
+            _.each($(el).find('a[accesskey]:visible'), function ($a) {
+                $($a).attr('accesskey').toLowerCase()
+                replaceHtml($($a).attr('accesskey').toUpperCase(), $a);
+                keys.push($($a).attr('accesskey').toUpperCase())
+            });
+        }
+    });
+
+    // sidebar menu
+    _.each($hedaer.find('.oe_view_manager_sidebar').find('button[accesskey]'), function (el) {
+        $(el).attr('accesskey', $(el).attr('accesskey').toLowerCase());
+        replaceHtml($(el).attr('accesskey').toUpperCase(), el);
+        keys.push($(el).attr('accesskey').toUpperCase());
+    });
+
+    this.keys = keys;
+
+    // accesskey for body
+    _.each ($('.oe_view_manager_body > div:not(.oe_view_manager_view_kanban)'), function(el) {
+        if ($(el).css('display') == 'block') {
+            _.each($(el).find('button:not(.oe_form_invisible):visible'), function (el) {
+                var index = 0;
+                if (!$(el).hasClass('oe_follower') && !$(el).hasClass('oe_link') && !$(el).hasClass('oe_i')) {
+                    key = $(el).text().replace(/\s/g, '').charAt(0).toUpperCase();
+                    if (key)
+                        addkey(key, el, index);
+                }
+            });
+        }
+    });
+}
+function unhighlightAccessKeys() {
+    $('span.highlight').removeClass('highlight');
+    $('li.highlight').removeClass('highlight');
+}
 instance.web.Notification =  instance.web.Widget.extend({
     template: 'Notification',
     init: function() {

=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2014-01-31 00:52:02 +0000
+++ addons/web/static/src/js/view_form.js	2014-02-03 06:03:15 +0000
@@ -359,6 +359,31 @@
             }
             self.$el.add(self.$buttons).removeClass('oe_form_dirty');
             self.autofocus();
+
+            // shortcut for pager (on left arrow - right arrow)
+            $(document).bind('keydown', 'left', function (event) {
+                event.stopPropagation();
+                event.stopImmediatePropagation();
+                switch (event.data) {
+                    case 'left':
+                        self.dataset.previous();
+                        break;
+                }
+                var def = self.reload();
+                self.trigger('pager_action_executed');
+                return def;
+            }).bind('keydown', 'right', function(event) {
+                event.stopPropagation();
+                event.stopImmediatePropagation();
+                switch (event.data) {
+                    case 'right':
+                        self.dataset.next();
+                        break;
+                }
+                var def = self.reload();
+                self.trigger('pager_action_executed');
+                return def;
+            });
         });
     },
     /**

=== modified file 'addons/web/static/src/xml/base.xml'
--- addons/web/static/src/xml/base.xml	2014-01-14 13:39:53 +0000
+++ addons/web/static/src/xml/base.xml	2014-02-03 06:03:15 +0000
@@ -583,7 +583,7 @@
     <div class="oe_sidebar">
         <t t-foreach="widget.sections" t-as="section">
             <div class="oe_form_dropdown_section">
-                <button class="oe_dropdown_toggle oe_dropdown_arrow">
+                <button class="oe_dropdown_toggle oe_dropdown_arrow" t-att-accesskey="section.label[0]">
                     <t t-if="section.name == 'files'" t-raw="widget.items[section.name].length || ''"/>
                     <t t-esc="section.label"/>
                 </button>
@@ -710,7 +710,7 @@
 <t t-name="ListView.buttons">
     <div class="oe_list_buttons">
     <t t-if="!widget.no_leaf and widget.options.action_buttons !== false and widget.options.addable and widget.is_action_enabled('create')">
-        <button type="button" class="oe_button oe_list_add oe_highlight">
+        <button type="button" class="oe_button oe_list_add oe_highlight" accesskey="c">
             <t t-esc="widget.options.addable"/>
         </button>
     </t>
@@ -791,15 +791,15 @@
         <span class="oe_form_buttons_view">
             <!-- required for the bounce effect on button -->
             <div t-if="widget.is_action_enabled('edit')" style="display: inline-block;">
-                <button type="button" class="oe_button oe_form_button_edit" accesskey="E">Edit</button>
+                <button type="button" class="oe_button oe_form_button_edit" accesskey="e">Edit</button>
             </div>
             <button t-if="widget.is_action_enabled('create')"
-                type="button" class="oe_button oe_form_button_create" accesskey="C">Create</button>
+                type="button" class="oe_button oe_form_button_create" accesskey="c">Create</button>
         </span>
         <span class="oe_form_buttons_edit">
-            <button type="button" class="oe_button oe_form_button_save oe_highlight" accesskey="S">Save</button>
+            <button type="button" class="oe_button oe_form_button_save oe_highlight" accesskey="s">Save</button>
             <span class="oe_fade">or</span>
-            <a href="#" class="oe_bold oe_form_button_cancel" accesskey="D">Discard</a>
+            <a href="#" class="oe_bold oe_form_button_cancel" accesskey="d">Discard</a>
         </span>
     </t>
 </div>

=== modified file 'addons/web_kanban/static/src/xml/web_kanban.xml'
--- addons/web_kanban/static/src/xml/web_kanban.xml	2013-05-17 05:41:10 +0000
+++ addons/web_kanban/static/src/xml/web_kanban.xml	2014-02-03 06:03:15 +0000
@@ -15,7 +15,7 @@
 <div t-name="KanbanView.buttons" class="oe_kanban_buttons">
     <t t-if="widget.options.action_buttons !== false">
         <t t-if="widget.is_action_enabled('create')">
-            <button type="button" class="oe_kanban_button_new oe_highlight">
+            <button type="button" class="oe_kanban_button_new oe_highlight" accesskey="c">
                 <t t-esc="widget.options.create_text || _t('Create')"/>
             </button>
             <span class="oe_alternative" style="display: none">