From 54159094172fdc3e75b787ea3e5012d1963992ab Mon Sep 17 00:00:00 2001
From: Derek Hageman <hageman@inthat.cloud>
Date: Wed, 4 Aug 2021 11:40:12 -0600
Subject: [PATCH] Fix cartesian hover toggle

The hover toggle button couldn't be enabled on cartesian plots because
the later setup completely overwrote the initial allowed groups.  So
just make it add the cartesian buttons instead of replacing the groups
completely.
---
 src/components/modebar/manage.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/modebar/manage.js b/src/components/modebar/manage.js
index 718a790f5e3..b11f97643e5 100644
--- a/src/components/modebar/manage.js
+++ b/src/components/modebar/manage.js
@@ -183,7 +183,7 @@ function getButtonGroups(gd) {
     // regardless of what other types are on the plot, since they'll all
     // just treat any truthy hovermode as 'closest'
     if(hasCartesian) {
-        hoverGroup = ['toggleSpikelines', 'hoverClosestCartesian', 'hoverCompareCartesian'];
+        hoverGroup.push('toggleSpikelines', 'hoverClosestCartesian', 'hoverCompareCartesian');
     }
     if(hasNoHover(fullData) || hasUnifiedHoverLabel) {
         hoverGroup = [];