Skip to content

Fix chart tooltip links on hover - #1447

Open
lovasoa wants to merge 2 commits into
mainfrom
fix-chart-tooltip-link-hover
Open

Fix chart tooltip links on hover#1447
lovasoa wants to merge 2 commits into
mainfrom
fix-chart-tooltip-link-hover

Conversation

@lovasoa

@lovasoa lovasoa commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Keep linked chart tooltips open while moving from a data point to the tooltip.
  • Close the tooltip after leaving it.
  • Cover Gantt, bar, and scatter linked-tooltip handoffs.

Testing

  • npm test
  • SQLPAGE_FIXTURE_BASE=http://127.0.0.1:18081 SQLPAGE_TEST_BASE=http://127.0.0.1:18081 npx playwright test fixtures/chart/test.ts --project=fixtures --grep 'interactive data point link|keeps a data point link open'\n\nThe line fixture does not currently surface its tooltip link, and the pie fixture places its linked tooltip outside the viewport; those are separate from the hover-close issue.

@lovasoa
lovasoa force-pushed the fix-chart-tooltip-link-hover branch from cdb37ee to 13db046 Compare September 11, 2026 16:38
@lovasoa
lovasoa force-pushed the fix-chart-tooltip-link-hover branch from 13db046 to 8e3db69 Compare September 11, 2026 17:23

@81reap 81reap left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so this does work, but it feels very inconsistent. to save ourselves the trouble we should consider using a browser native API like https://developer.mozilla.org/en-US/docs/Web/HTML/Reference/Global_attributes/popover#hint

it's supported everywhere but Safari, where its available in pre-release https://developer.mozilla.org/en-US/docs/Web/HTML/Reference/Global_attributes/popover#browser_compatibility

maybe even an upstream fix to apexcharts? seems like this is a repeated issue

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.

2 participants