From a5cb0866244438fd6afef399b03a097e8267c9f2 Mon Sep 17 00:00:00 2001
From: Vishmita <vja@tinyerp.com>
Date: Tue, 6 Nov 2012 15:16:04 +0530
Subject: [PATCH] [IMP]Improve code for keyboard shortcuts.

bzr revid: vja@tinyerp.com-20121106094604-q2aiwv655vajdebj
---
 addons/web/static/src/css/base.css    |  4 ++++
 addons/web/static/src/css/base.sass   | 12 ++++++++----
 addons/web/static/src/js/view_form.js |  4 +++-
 3 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/addons/web/static/src/css/base.css b/addons/web/static/src/css/base.css
index 059b51b183e..91793947836 100644
--- a/addons/web/static/src/css/base.css
+++ b/addons/web/static/src/css/base.css
@@ -475,6 +475,10 @@
   max-width: 180px;
   max-height: 180px;
 }
+.openerp .oe_view_manager_current a.accessactive {
+   display:inline-block;
+   text-decoration:none;
+}
 .openerp .oe_view_manager_current .accessactive:first-letter{
   text-decoration:underline;
   text-shadow: 0 1px 1px rgba(0, 0, 0, 0.2);
diff --git a/addons/web/static/src/css/base.sass b/addons/web/static/src/css/base.sass
index b96321a78f4..a1bb16d6856 100644
--- a/addons/web/static/src/css/base.sass
+++ b/addons/web/static/src/css/base.sass
@@ -437,10 +437,14 @@ $sheet-padding: 16px
     .oe_image_medium > img
         max-width: 180px
         max-height: 180px
-    .oe_view_manager_current .accessactive:first-letter
-        text-decoration: underline
-        text-shadow: 0 1px 1px rgba(0, 0, 0, 0.2)
-        font-weight: bold
+    .oe_view_manager_current
+        a.accessactive
+            display: inline-block
+            text-decoration: none
+        .accessactive:first-letter
+            text-decoration: underline
+            text-shadow: 0 1px 1px rgba(0, 0, 0, 0.2)
+            font-weight: bold
     .oe_button.oe_link
         @include reset()
         img
diff --git a/addons/web/static/src/js/view_form.js b/addons/web/static/src/js/view_form.js
index cea026397ea..67dbc4b0acf 100644
--- a/addons/web/static/src/js/view_form.js
+++ b/addons/web/static/src/js/view_form.js
@@ -222,8 +222,10 @@ instance.web.FormView = instance.web.View.extend(instance.web.form.FieldManagerM
     },
     widgetAccesskey:function(){
         $(document).keydown(function(e){
-            if(e.keyCode === $.ui.keyCode.ALT) {
+            if(e.keyCode === $.ui.keyCode.ALT || e.keyCode === $.ui.keyCode.SHIFT && e.altKey) {
                 $("[accesskey]").addClass('accessactive');
+            }else {
+                $("[accesskey]").removeClass('accessactive');
             }
             e.stopPropagation();
         }).keyup(function(e){