Skip to content

fix(chartcuterie): Stack grouped bar charts in unfurls#113177

Merged
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1536-bar-chart-not-stacked
Apr 16, 2026
Merged

fix(chartcuterie): Stack grouped bar charts in unfurls#113177
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1536-bar-chart-not-stacked

Conversation

@DominikB2014
Copy link
Copy Markdown
Contributor

Grouped bar charts rendered by chartcuterie for Slack Explore/Dashboards unfurls were not stacking, so each group rendered as its own side-by-side bar instead of accumulating into one column per x-tick. This didn't match how the equivalent charts render in Explore or Dashboards.

Pass a shared stack key to the plottable when rendering grouped series, matching Explore's chartVisualization.tsx which also passes stack: 'all'. ECharts stacks every series that shares the same stack string, so all series in a given chart accumulate into a single stack per x-tick.

Only the grouped branch needs the change — the single-series branch renders one series and has nothing to stack.

Refs DAIN-1536

Bar charts rendered by chartcuterie for Slack unfurls were not stacking
grouped series, unlike the equivalent charts in Explore and Dashboards.
Pass a shared stack key to the plottable when rendering grouped series
so Bars plottables stack under a single column per x-tick.

Refs DAIN-1536
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@linear-code
Copy link
Copy Markdown

linear-code Bot commented Apr 16, 2026

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 16, 2026
@DominikB2014 DominikB2014 marked this pull request as ready for review April 16, 2026 16:10
@DominikB2014 DominikB2014 requested a review from a team as a code owner April 16, 2026 16:10
@DominikB2014 DominikB2014 merged commit 79b2856 into master Apr 16, 2026
66 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/dain-1536-bar-chart-not-stacked branch April 16, 2026 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants