Skip to content

fix(TooltipContext): correct bisect-band hit detection#806

Merged
techniq merged 1 commit intotechniq:nextfrom
blowindblo:fix/bisect-band-padding
Mar 26, 2026
Merged

fix(TooltipContext): correct bisect-band hit detection#806
techniq merged 1 commit intotechniq:nextfrom
blowindblo:fix/bisect-band-padding

Conversation

@blowindblo
Copy link

When using the tooltip with mode="bisect-band", pointer coordinates were not adjusted for chart padding. As a result, the tooltip could snap to the wrong data point whenever left or top padding was present.

bisect-x and bisect-y modes already subtract ctx.padding.left and ctx.padding.top correctly. This PR applies the same offsets to bisect-band.

@changeset-bot
Copy link

changeset-bot bot commented Mar 26, 2026

🦋 Changeset detected

Latest commit: 7d71856

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
layerchart Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 26, 2026

Open in StackBlitz

npm i https://pkg.pr.new/layerchart@806

commit: 7d71856

@github-actions
Copy link
Contributor

github-actions bot commented Mar 26, 2026

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
layerchart ✅ Ready (View Log) Visit Preview 7d71856

Copy link
Owner

@techniq techniq left a comment

Choose a reason for hiding this comment

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

Thanks @blowindblo!

@techniq techniq merged commit 0d35cae into techniq:next Mar 26, 2026
7 checks passed
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