Merge pull request #4482 from rosco54/Issue_4474

Stacked column totals dataLabel position fix (category X-axis).
apexcharts · May 24, 2024 · 8b34013 · 8b34013
2 parents c677e1a + f403059
commit 8b34013
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 12 deletions.
diff --git a/src/charts/BarStacked.js b/src/charts/BarStacked.js
@@ -518,14 +518,12 @@ class BarStacked extends Bar {
       elSeries,
     })
 
-    x = x + xDivision
-
     return {
       pathTo: paths.pathTo,
       pathFrom: paths.pathFrom,
       goalY: this.barHelpers.getGoalValues('y', null, zeroH, i, j),
       barXPosition,
-      x: w.globals.isXNumeric ? x - xDivision : x,
+      x: w.globals.isXNumeric ? x : x + xDivision,
       y,
     }
   }

diff --git a/src/charts/common/bar/DataLabels.js b/src/charts/common/bar/DataLabels.js
@@ -328,16 +328,14 @@ export default class BarDataLabels {
           ADDITIONAL_OFFY
       }
 
+      // width divided into equal parts
+      let xDivision = w.globals.gridWidth / w.globals.dataPoints
+
       totalDataLabelsX =
-        totalDataLabelsBcx +
-        (w.globals.isXNumeric
-          ? barWidth * (w.globals.barGroups.length - 1) - barWidth / 2
-          : -(
-              barWidth * w.globals.barGroups.length -
-              barWidth / 2 -
-              strokeWidth * 2
-            )) +
-        barTotalDataLabelsConfig.offsetX
+          totalDataLabelsBcx
+        + barWidth * (w.globals.barGroups.length - 0.5)
+        - (w.globals.isXNumeric ? barWidth : xDivision)
+        + barTotalDataLabelsConfig.offsetX
     }
 
     if (!w.config.chart.stacked) {