From ed2f14784dde25ff372dcf31ac1b2a3d40634972 Mon Sep 17 00:00:00 2001 From: Mohammed Shekha Date: Thu, 28 May 2020 05:51:29 +0000 Subject: [PATCH] [IMP] web: cursor pointer on hover graph legend Before this commit: hovering on graph legend shows cursor default here hovering on legend should display cursor pointer, so user can easily understand it is clickable element After this commit, hovering on graph legend will display cursor pointer. task-2263703 closes odoo/odoo#52030 Signed-off-by: Aaron Bohy (aab) --- addons/web/static/src/js/views/graph/graph_renderer.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/addons/web/static/src/js/views/graph/graph_renderer.js b/addons/web/static/src/js/views/graph/graph_renderer.js index c179a08216e..4def49a6d7e 100644 --- a/addons/web/static/src/js/views/graph/graph_renderer.js +++ b/addons/web/static/src/js/views/graph/graph_renderer.js @@ -1071,6 +1071,8 @@ return AbstractRenderer.extend({ * @param {Object} legendItem */ _onlegendTooltipHover: function (e, legendItem) { + // set cursor pointer on hover of legend + e.target.style.cursor = 'pointer'; // The string legendItem.text is an initial segment of legendItem.fullText. // If the two coincide, no need to generate a tooltip. // If a tooltip for the legend already exists, it is already good and don't need @@ -1102,7 +1104,9 @@ return AbstractRenderer.extend({ * * @private */ - _onLegendTootipLeave: function () { + _onLegendTootipLeave: function (e) { + // remove cursor style pointer on mouseleave from legend + e.target.style.cursor = ""; if (this.$legendTooltip) { this.$legendTooltip.remove(); this.$legendTooltip = null;