fix split mode hover (remove outdated logic)

emilykl · emilykl · commit 6c901b24e34e · 2025-11-12T18:34:36.000-05:00
diff --git a/src/traces/ohlc/hover.js b/src/traces/ohlc/hover.js
@@ -99,17 +99,6 @@ function hoverSplit(pointData, xval, yval, hovermode) {
     // skip the rest (for this trace) if we didn't find a close point
     if(!closestPoint) return [];
 
-    var cdIndex = closestPoint.index;
-    var di = cd[cdIndex];
-    var hoverinfo = di.hi || trace.hoverinfo;
-    var hoverParts = hoverinfo.split('+');
-    var isAll = hoverinfo === 'all';
-    var hasY = isAll || hoverParts.indexOf('y') !== -1;
-
-    // similar to hoverOnPoints, we return nothing
-    // if all or y is not present.
-    if(!hasY) return [];
-
     var attrs = ['high', 'open', 'close', 'low'];
 
     // several attributes can have the same y-coordinate. We will
@@ -168,15 +157,14 @@ function hoverOnPoints(pointData, xval, yval, hovermode) {
     var hoverinfo = di.hi || trace.hoverinfo || '';
     var hoverParts = hoverinfo.split('+');
     var isAll = hoverinfo === 'all';
-    var hasY = isAll || hoverParts.indexOf('y') !== -1;
     var hasText = isAll || hoverParts.indexOf('text') !== -1;
 
-    var textParts = hasY ? [
+    var textParts = [
         getLabelLine('open'),
         getLabelLine('high'),
         getLabelLine('low'),
         getLabelLine('close') + '  ' + DIRSYMBOL[dir]
-    ] : [];
+    ];
     if(hasText) fillText(di, trace, textParts);
 
     // don't make .yLabelVal or .text, since we're managing hoverinfo
