Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Brush integration (simplified charts) #346

Open
8 of 14 tasks
techniq opened this issue Jan 27, 2025 · 0 comments
Open
8 of 14 tasks

Brush integration (simplified charts) #346

techniq opened this issue Jan 27, 2025 · 0 comments

Comments

@techniq
Copy link
Owner

techniq commented Jan 27, 2025 •
edited
Loading

Similar to the recent Legend integration (#257), we should integrate Brush into all simplified charts

Use cases

  • Integrated
    • <AreaChart brush={{ mode: 'integrated }}>
  • Selection
    • <AreaChart brush={{ mode: 'selection }}>
  • Separate (clip data) (second chart below)
    • <AreaChart brush={{ mode: 'separate }}>
  • Separate (filter data) (second chart below)
    • <AreaChart brush={{ mode: 'separate-filter }}>
  • Set initial value
  • Sync multiple charts

Charts
techniq added a commit that referenced this issue Jan 27, 2025
@techniq
feat(AreaChart|LineChart|ScatterChart): Add Brush integration. Issue …
befda7d 
…Brush integration (simplified charts) #346
techniq added a commit that referenced this issue Jan 27, 2025
@techniq
Brush integration (#348)
c35bc28 
* feat(ChartClipPath|CircleClipPath|RectClipPath): Support passing `disabled` to underlying `ClipPath`

* feat: Support passing `props={{ tooltip: { context: ... } }}` for all simplified charts (still support `tooltip={...}`) and restore ability to pass `tooltip={false}` to disable tooltip for all simplified charts

* feat(AreaChart|LineChart|ScatterChart): Add Brush integration.  Issue Brush integration (simplified charts) #346

* fix(AreaChart|LineChart): Fade other series highlight points when hovering individual point

* fix(Brush): Use `DomainType` for x/y domain to fix type check

* refactor(Brush): Use common `asAny()` workaround
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@techniq