shared tooltip fix in bar chart; fixes #3439

Author: junedchhipa

src/charts/Bar.js

@@ -139,7 +139,7 @@ class Bar {
         w.config.yaxis[this.yaxisIndex] &&
         w.config.yaxis[this.yaxisIndex].reversed
 
-      let initPositions = this.barHelpers.initialPositions()
+      let initPositions = this.barHelpers.initialPositions(realIndex)
 
       y = initPositions.y
       barHeight = initPositions.barHeight

src/charts/BoxCandleStick.js

@@ -74,7 +74,7 @@ class BoxCandleStick extends Bar {
         translationsIndex = realIndex
       }
 
-      let initPositions = this.barHelpers.initialPositions()
+      let initPositions = this.barHelpers.initialPositions(realIndex)
 
       y = initPositions.y
       barHeight = initPositions.barHeight

src/charts/RangeBar.js

@@ -59,7 +59,7 @@ class RangeBar extends Bar {
         translationsIndex = realIndex
       }
 
-      let initPositions = this.barHelpers.initialPositions()
+      let initPositions = this.barHelpers.initialPositions(realIndex)
 
       y = initPositions.y
       zeroW = initPositions.zeroW

src/charts/common/bar/Helpers.js

@@ -50,7 +50,7 @@ export default class Helpers {
     }
   }
 
-  initialPositions() {
+  initialPositions(realIndex) {
     let w = this.w
     let x, y, yDivision, xDivision, barHeight, barWidth, zeroH, zeroW
 
@@ -136,9 +136,19 @@ export default class Helpers {
           ? this.barCtx.baseLineY[this.barCtx.translationsIndex] * 2
           : 0)
 
-      x =
-        w.globals.padHorizontal +
-        Utils.noExponents(xDivision - barWidth * this.barCtx.seriesLen) / 2
+      if (w.globals.isXNumeric) {
+        const xForNumericX = this.barCtx.getBarXForNumericXAxis({
+          x,
+          j: 0,
+          realIndex,
+          barWidth,
+        })
+        x = xForNumericX.x
+      } else {
+        x =
+          w.globals.padHorizontal +
+          Utils.noExponents(xDivision - barWidth * this.barCtx.seriesLen) / 2
+      }
     }
 
     w.globals.barHeight = barHeight

src/modules/tooltip/Utils.js

@@ -172,16 +172,18 @@ export default class Utils {
       const xArr = Xarrays[i]
       const yArr = Yarrays[i]
 
-      // Ensure both xArr and yArr have equal lengths and valid data
       const len = Math.min(xArr.length, yArr.length)
 
       for (let j = 0; j < len; j++) {
         const xVal = xArr[j]
-        const yVal = yArr[j]
-
         const distX = hoverX - xVal
-        const distY = hoverY - yVal
-        const dist = Math.sqrt(distX * distX + distY * distY)
+        let dist = Math.sqrt(distX * distX)
+
+        if (!w.globals.allSeriesHasEqualX) {
+          const yVal = yArr[j]
+          const distY = hoverY - yVal
+          dist = Math.sqrt(distX * distX + distY * distY)
+        }
 
         if (dist < closestDist) {
           closestDist = dist