Remove use of global ApexCharts class

[SmithKy3]

New Pull Request

The setupBrushHandler method of the Core class relies on the ApexCharts class' static getChartByID method. It calls this static method on the ApexCharts class directly without importing it currently.

This is fine if consuming apexcharts via
CDN / script tag as that places the exported ApexCharts class in the global scope, but it becomes an issue when users want to consume apexcharts via a package manager.

Update the setupBrushHandler method to call ApexCharts.getChartByID the Core class' ctx field instead.

Fixes apexcharts/react-apexcharts#666

Type of change

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• • e2e tests passed locally
• • I haven't been able to verify that the unit test suite passes because they seem to be broken. I get the getBBox is not a function error when I run them, which I can see has been occurring in your Node.js CI action. I'm not sure if that's related to the browser API or one of your internal utils but let me know if you want me to rebase / merge changes in once that's fixed and run the unit tests.
• My branch is up to date with any changes from the main branch