Skip to content

fix(chartcuterie): Format y-axis labels in Explore unfurl charts#112719

Merged
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1496-y-axis-label-formatting-doesnt-match-explore
Apr 10, 2026
Merged

fix(chartcuterie): Format y-axis labels in Explore unfurl charts#112719
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1496-y-axis-label-formatting-doesnt-match-explore

Conversation

@DominikB2014
Copy link
Copy Markdown
Contributor

Explore unfurl charts were showing raw numbers on the y-axis (e.g. 300,000)
instead of abbreviated values (300K), and duration fields were missing units
(showing 5 instead of 5ms).

Adds a y-axis formatter using formatYAxisValue — the same function the
Explore page uses — which reads valueType and valueUnit from the
timeseries metadata to format labels appropriately for integers, durations,
sizes, rates, and percentages.

Fixes DAIN-1496

@linear-code
Copy link
Copy Markdown

linear-code bot commented Apr 10, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 10, 2026
@DominikB2014 DominikB2014 marked this pull request as ready for review April 10, 2026 19:30
Comment thread static/app/chartcuterie/explore.tsx
Y-axis labels now use the same formatting as the Explore page:
- Large numbers are abbreviated (300,000 → 300K)
- Duration values show units (5 → 5ms)
- Sizes, rates, percentages are formatted appropriately

Uses formatYAxisValue with the timeseries meta (valueType, valueUnit)
to match the Explore page's formatter behavior.

Fixes DAIN-1496
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@DominikB2014 DominikB2014 force-pushed the dominikbuszowiecki/dain-1496-y-axis-label-formatting-doesnt-match-explore branch from 9d0b4c6 to e831ea0 Compare April 10, 2026 19:41
@DominikB2014 DominikB2014 merged commit 018fdaa into master Apr 10, 2026
61 of 65 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/dain-1496-y-axis-label-formatting-doesnt-match-explore branch April 10, 2026 19:51
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