From 80a0cbba19d232c0c71d7951db707924ef9ff382 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alexandre=20D=C3=ADaz?= <alexandre.diaz@tecnativa.com>
Date: Tue, 14 Apr 2020 17:05:09 +0200
Subject: [PATCH] [IMP] web_tree_many2one_clickable: Don't break keyboard
 navigation

---
 .../src/css/web_tree_many2one_clickable.scss  | 21 ++++++++++++-------
 .../src/js/web_tree_many2one_clickable.js     | 15 +++++++------
 2 files changed, 21 insertions(+), 15 deletions(-)

diff --git a/web_tree_many2one_clickable/static/src/css/web_tree_many2one_clickable.scss b/web_tree_many2one_clickable/static/src/css/web_tree_many2one_clickable.scss
index de3216d00..720183aeb 100644
--- a/web_tree_many2one_clickable/static/src/css/web_tree_many2one_clickable.scss
+++ b/web_tree_many2one_clickable/static/src/css/web_tree_many2one_clickable.scss
@@ -1,10 +1,17 @@
 td.o_many2one_cell {
-  a {
-    margin-left: 0.5em;
-    visibility: hidden;
-  }
+    a {
+        color: $o-main-text-color !important;
+        &:hover {
+            color: $o-main-text-color !important;
+        }
 
-  &:hover a {
-    visibility: visible;
-  }
+        .many2one_clickable {
+            margin-left: 0.5em;
+            visibility: hidden;
+        }
+    }
+
+    &:hover a.many2one_clickable {
+        visibility: visible;
+    }
 }
diff --git a/web_tree_many2one_clickable/static/src/js/web_tree_many2one_clickable.js b/web_tree_many2one_clickable/static/src/js/web_tree_many2one_clickable.js
index 9a057e85d..b201901b3 100644
--- a/web_tree_many2one_clickable/static/src/js/web_tree_many2one_clickable.js
+++ b/web_tree_many2one_clickable/static/src/js/web_tree_many2one_clickable.js
@@ -32,19 +32,18 @@ odoo.define('web_tree_many2one_clickable.many2one_clickable', function (require)
             var self = this;
 
             if (!this.noOpen && this.value) {
-                // Replace '<a>' element
-                this.$el.removeClass('o_form_uri');
-                this.$el = $('<span/>', {
-                    html: this.$el.html(),
-                    class: this.$el.attr('class') + ' o_field_text',
-                    name: this.$el.attr('name'),
-                });
+                // Disable 'click' events
+                this.$el.off('click');
+                this.$el.on('click', function (ev) {
+                    ev.preventDefault();
+                })
 
                 // Append button
                 var $a = $('<a/>', {
                     href: '#',
                     class: 'o_form_uri btn btn-sm btn-secondary' +
-                           ' fa fa-angle-double-right',
+                           ' fa fa-angle-double-right many2one_clickable',
+                    tabindex: '-1',
                 }).on('click', function (ev) {
                     ev.preventDefault();
                     ev.stopPropagation();