Merge pull request #5879 from Sizurka/fix-hover-toggle

archmoj · web-flow · commit fb78b32ca14c · 2024-07-13T15:41:08.000-04:00
Fix cartesian hover toggle
diff --git a/src/components/modebar/manage.js b/src/components/modebar/manage.js
@@ -185,7 +185,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 = [];

Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,7 @@ function getButtonGroups(gd) {`
`185`	`185`	`// regardless of what other types are on the plot, since they'll all`
`186`	`186`	`// just treat any truthy hovermode as 'closest'`
`187`	`187`	`if(hasCartesian) {`
`188`		`- hoverGroup = ['toggleSpikelines', 'hoverClosestCartesian', 'hoverCompareCartesian'];`
	`188`	`+ hoverGroup.push('toggleSpikelines', 'hoverClosestCartesian', 'hoverCompareCartesian');`
`189`	`189`	`}`
`190`	`190`	`if(hasNoHover(fullData) \|\| hasUnifiedHoverLabel) {`
`191`	`191`	`hoverGroup = [];`