From 4bff0c04bb422042b86ca43622ec5a7dcf78bdf2 Mon Sep 17 00:00:00 2001
From: Damien Bouvy <dbo@odoo.com>
Date: Mon, 15 May 2023 14:04:17 +0000
Subject: [PATCH] [IMP] web: remove hover effect on list columns

The hover effect is rather strong and has low usefulness (the "chevron"
icon still appears) and can interfere with the current "sorting column"
decoration. We can safely remove it.

X-original-commit: f70fcbc8d1971c68546242c59caaae7f8481acf2
Part-of: odoo/odoo#122111
---
 addons/web/static/src/views/list/list_renderer.js  | 14 --------------
 addons/web/static/src/views/list/list_renderer.xml |  2 --
 2 files changed, 16 deletions(-)

diff --git a/addons/web/static/src/views/list/list_renderer.js b/addons/web/static/src/views/list/list_renderer.js
index 9920069aa5a..4099442a1c9 100644
--- a/addons/web/static/src/views/list/list_renderer.js
+++ b/addons/web/static/src/views/list/list_renderer.js
@@ -1839,20 +1839,6 @@ export class ListRenderer extends Component {
         });
     }
 
-    /**
-     * Handles the :hover effect on sortable column headers
-     *
-     * @private
-     * @param {MouseEvent} ev
-     */
-    onHoverSortColumn(ev, column) {
-        if (this.props.list.orderBy.length && this.props.list.orderBy[0].name === column.name) {
-            return;
-        } else if (this.isSortable(column) && column.widget !== "handle") {
-            ev.target.classList.toggle("table-active", ev.type == "mouseenter");
-        }
-    }
-
     onColumnTitleMouseUp() {
         if (this.resizing) {
             this.preventReorder = true;
diff --git a/addons/web/static/src/views/list/list_renderer.xml b/addons/web/static/src/views/list/list_renderer.xml
index 56206fed12b..5d5df5ba627 100644
--- a/addons/web/static/src/views/list/list_renderer.xml
+++ b/addons/web/static/src/views/list/list_renderer.xml
@@ -20,8 +20,6 @@
                             <th t-if="column.type === 'field'"
                                 t-att-data-name="column.name"
                                 t-att-class="getColumnClass(column) + ' opacity-trigger-hover'"
-                                t-on-mouseenter="ev => this.onHoverSortColumn(ev, column)"
-                                t-on-mouseleave="ev => this.onHoverSortColumn(ev, column)"
                                 t-on-mouseup="onColumnTitleMouseUp"
                                 t-on-click="() => this.onClickSortColumn(column)"
                                 t-on-keydown.synthetic="(ev) => this.onCellKeydown(ev)"