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.

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

Fix RowChart X-Axis ticks formatter #43783

Merged

Conversation

JesseSDevaney
Copy link
Contributor

Remove scaling from x-axis tick values like we did for y-axis tick values in ECharts charts


Before
No scaling ✔️
Screenshot from 2024-06-06 15-53-31

With scaling
Screenshot from 2024-06-06 15-54-21

Static-viz with scaling
Screenshot from 2024-06-06 15-57-09

After

With scaling ✔️
Screenshot from 2024-06-06 15-56-12

Static-viz with scaling ✔️
Screenshot from 2024-06-06 15-56-59


Note: This was meant to be handled by #43668 but the commit for the fix did not get pushed to the branch before it got merged.

- remove scaling from x-axis tick values like we did for y-axis tick values in ECharts charts
@JesseSDevaney JesseSDevaney added backport Automatically create PR on current release branch on merge .Team/DashViz Dashboard and Viz team labels Jun 6, 2024
@JesseSDevaney JesseSDevaney requested a review from a team June 6, 2024 20:15
@JesseSDevaney JesseSDevaney self-assigned this Jun 6, 2024
Copy link

github-actions bot commented Jun 6, 2024

Codenotify: Notifying subscribers in CODENOTIFY files for diff e3f166b...a6caf7c.

Notify File(s)
@alxnddr frontend/src/metabase/visualizations/visualizations/RowChart/utils/format.ts

Copy link

replay-io bot commented Jun 6, 2024

Status Complete ↗︎
Commit a6caf7c
Results
⚠️ 4 Flaky
2595 Passed

@JesseSDevaney JesseSDevaney enabled auto-merge (squash) June 6, 2024 21:10
@JesseSDevaney JesseSDevaney merged commit f54e3d5 into master Jun 6, 2024
112 checks passed
@JesseSDevaney JesseSDevaney deleted the row-charts-x-axis-formatting-applies-scaling-twice branch June 6, 2024 22:26
Copy link

github-actions bot commented Jun 6, 2024

@JesseSDevaney Did you forget to add a milestone to the issue for this PR? When and where should I add a milestone?

github-automation-metabase pushed a commit that referenced this pull request Jun 6, 2024
* fix RowChart XAxis ticks formatter

- remove scaling from x-axis tick values like we did for y-axis tick values in ECharts charts

* add loki variant
github-automation-metabase added a commit that referenced this pull request Jun 6, 2024
* fix RowChart XAxis ticks formatter

- remove scaling from x-axis tick values like we did for y-axis tick values in ECharts charts

* add loki variant

Co-authored-by: Jesse Devaney <22608765+JesseSDevaney@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Automatically create PR on current release branch on merge .Team/DashViz Dashboard and Viz team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Row Chart x-axis formatting applies column metric scaling twice
2 participants