Skip to content

State improvements#663

Open
techniq wants to merge 141 commits intodocs-v2from
state-refactor
Open

State improvements#663
techniq wants to merge 141 commits intodocs-v2from
state-refactor

Conversation

@techniq
Copy link
Owner

@techniq techniq commented Oct 19, 2025 •
edited
Loading

Goals

  • Integrate simplified charts
    • snippets / default impl
    • simplified setup
    • series state
  • External control / sync
    • Set initial highlightSeriesKey
    • Save/restore state to localStorage / query string / etc
  • Improve tooltip data (multi-series with/without data, etc)
    • currently expose tooltipState.data and tooltipState.payload
  • Chart
    • Add series prop
    • Add orientation to Chart (if needed)?
      • Added valueAxis due to inverted orientation meaning in LineChart/BarChart
  • ChartState / context
    • Expose seriesState on context
    • Simplify HighlightKey
  • Rename ChartState geoContext / brushContext / tooltipContext to geoState / etc
  • Rename <GeoContext bind:geoContext> to <GeoContext bind:state> (and BrushContext, TooltipContext, TransformContext)
  • Instantiate GeoState etc within Chart / ChartState?
    • Replace getGeoContext with getChartContext and use ctx.geo
      • Useful to keep getGeoContext for overrides such as a translucent globe
    • Do we need GeoContext / etc? (if it doesn't have any pointer events / elements)
      • Only when overriding (not within Chart. Simplified and renamed GeoContext to GeoProjection
    • Move $effect()logic in GeoContext to GeoState
  • Fix <BarChart seriesLayout="group"> regression (before this PR)
  • Remove createLegendProps() and handle directly since ctx.seriesState is available
  • Update ChartAnnotations to use ctx.seriesState
  • Update DefaultTooltip to use ctx.seriesState
  • GeoDebug usage
  • TransformDebug usage
  • GeoTile performance (local-only?)
  • Should TransformState handle pointer events onPointerMove, etc or within TransformContext
  • Rename tooltipContext props (Arc, GeoPath, etc) to tooltipState (or just tooltip)?
  • Remove tooltipMetaContext (consolidate payload, etc)
  • Bounds use case integration (reference)

Verify

To fix

related:

@changeset-bot
Copy link

changeset-bot bot commented Oct 19, 2025 •
edited
Loading

🦋 Changeset detected

Latest commit: c1a9c15

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

@techniq techniq changed the base branch from main to docs-v2 October 19, 2025 18:44
@github-actions
Copy link
Contributor

github-actions bot commented Oct 19, 2025 •
edited
Loading

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 19, 2025 •
edited
Loading

Open in StackBlitz

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

commit: c1a9c15

@github-actions github-actions bot requested a deployment to Preview January 20, 2026 12:50 Abandoned
@techniq techniq mentioned this pull request Jan 20, 2026
Closed
@techniq
Move resolveAccessor out of AreaChart/BarChart and into series.svel…
9b3e13a 
…te.ts, but will refine further into chart.svelte.ts soon.
@techniq techniq mentioned this pull request Jan 24, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment