Skip to content

feat(datagrid): split result charts into one panel per column value - #3593

Merged
datlechin merged 2 commits into
mainfrom
feat/chart-facets
Oct 19, 2026
Merged

datlechin merged 2 commits into
mainfrom
feat/chart-facets

Conversation

@datlechin

Copy link
Copy Markdown
Member

Why

A result chart drew one plot. Comparing the same chart across regions or products took one query per value or an export.

Change

  • Facet picker beside X, Y and Series: one chart per value, in a grid with shared axes and one legend. Up to 12 panels, in the query's order.
  • A series keeps its color, symbol and dash in every panel and through a re-sort or page turn.
  • Chart and map keep their drawing while a refresh, sort or page turn rebuilds it. The chart no longer shows the previous result's notices.
  • Empty text reads "Empty" in chart labels, NULL no longer merges with a literal "No value", and a picker whose column left the result shows its none item instead of going blank.

Verified

New projector, canvas, pane state and layout tests cover each fix. The canvas tests for series color and bar slots across panels fail with the explicit scales and slot order taken out. ResultChartSelectionTests and one ResultChartProjectorTests case only follow renamed types.

Screenshots

Rendered from the chart views with ImageRenderer (a sandboxed build has no chart license). The chart type control is left out: ImageRenderer cannot draw it.

Light Dark
Facet: None Facet None light Facet None dark
Facet: region Facet region light Facet region dark

Not in this PR

  • %d more and %d skipped have no French plural. Only 22 of 356 count strings do, so it needs one catalog pass.
  • Facet takes text, boolean, enum and set columns, not an integer like a year.

Fixes #3348

@datlechin
datlechin merged commit 6a104a9 into main Oct 19, 2026
17 checks passed
@datlechin
datlechin deleted the feat/chart-facets branch October 19, 2026 16:31
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.

Add faceting (small multiples) to result charts

1 participant