Skip to content

refactor(chart) :: import and use 3rd party ApexCharts types - #1516

Open
81reap wants to merge 1 commit into
81reap/ts-10-any-sqlpagefrom
81reap/ts-11-any-apexcharts
Open

81reap wants to merge 1 commit into
81reap/ts-10-any-sqlpagefrom
81reap/ts-11-any-apexcharts

Conversation

@81reap

@81reap 81reap commented Sep 30, 2026

Copy link
Copy Markdown
Collaborator

apexcharts.ts now has one implicit any, TooltipArgs keeps its w: any until upstream ApexCharts updates it's tooltip types.

Fixing this smoked out six workarounds ::

  1. options as ApexOptions hid that zaxis is not an ApexCharts option at all.
  2. yasix has to be normalized into an array and forwarded within bubbleTooltip. Now chartTooltip takes the axis titles directly.
  3. formatTooltipX returned a value where tooltip.x.formatter is declared to return a string.
  4. pie labels were (string | number)[] where ApexOptions wants string[].
  5. xaxis.tickAmount was assigned after the literal instead of inside it.
  6. the dataLabels formatters destructured an opts that ApexCharts declares optional, so they are now three named functions.

Stack created with GitHub Stacks CLI • Give Feedback 💬

`apexcharts.ts` now has one implicit any, `TooltipArgs` keeps its `w: any` until upstream ApexCharts updates it's tooltip types.

Fixing this smoked out six workarounds ::
1. `options as ApexOptions` hid that `zaxis` is not an ApexCharts option at all.
2. `yasix` has to be normalized into an array and forwarded within `bubbleTooltip`. Now `chartTooltip` takes the axis titles directly.
3. `formatTooltipX` returned a value where `tooltip.x.formatter` is declared to return a `string`.
4. pie `labels` were `(string | number)[]` where `ApexOptions` wants `string[]`.
5. `xaxis.tickAmount` was assigned after the literal instead of inside it.
6. the `dataLabels` formatters destructured an `opts` that ApexCharts declares optional, so they are now three named functions.
@81reap
81reap added this pull request to stack #1512 September 30, 2026 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant