Merge pull request #5201 from apexcharts/copilot/fix-line-chart-annotations-issue

fix: x-axis annotations missing on line/area/scatter charts (5.10.4 regression)

Author: junedchhipa

src/modules/annotations/Helpers.js

@@ -300,7 +300,11 @@ export default class Helpers {
       w.config.xaxis.convertedCatToNumeric &&
       w.labelData.categoryLabels.length
     ) {
-      x = w.labelData.categoryLabels.indexOf(String(x)) + 1
+      const strX = String(x)
+      x =
+        w.labelData.categoryLabels.findIndex(
+          (/** @type {any} */ l) => String(l) === strX,
+        ) + 1
     }
 
     const catIndex = w.labelData.labels

tests/unit/xaxis-annotations.spec.js

@@ -633,5 +633,30 @@ describe('XAxisAnnotations', () => {
       expect(labels[0].getAttribute('rel')).toBe('0')
       expect(labels[1].getAttribute('rel')).toBe('1')
     })
+
+    it('should render annotation line for numeric x value on line chart with simple array data (regression #5198)', () => {
+      // Regression test: in 5.10.4, annotations with numeric x values on line charts
+      // using simple array data (convertedCatToNumeric=true) were not rendered because
+      // getStringX() used indexOf(String(x)) which failed when categoryLabels are numbers.
+      chart = createChartWithOptions({
+        chart: { type: 'line' },
+        series: [{ data: [10, 20, 30, 40, 50] }],
+        annotations: {
+          xaxis: [
+            {
+              x: 3,
+              borderColor: '#ff0000',
+            },
+          ],
+        },
+      })
+
+      const line = chart.w.globals.dom.baseEl.querySelector(
+        '.apexcharts-xaxis-annotations line'
+      )
+      // The annotation line must be present (was missing in 5.10.4 due to regression)
+      expect(line).not.toBeNull()
+      expect(line.getAttribute('stroke')).toBe('#ff0000')
+    })
   })
 })