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 issue with bar with when data changes from zero values #1588

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

envex
Copy link
Collaborator

@envex envex commented Sep 21, 2023

Resolves https://github.com/Shopify/core-issues/issues/60729

Fixes issue when switching between data sets with data and data sets with all 0 values causing the bar width to be too small.

https://6062ad4a2d14cd0021539c1b-ikwtklqeiu.chromatic.com/?path=/story/polaris-viz-charts-barchart-playground--zero-data

@github-actions
Copy link

github-actions bot commented Sep 21, 2023

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
polaris-viz-core-cjs 61.33 KB (0%) 1.3 s (0%) 1.8 s (+15.15% 🔺) 3 s
polaris-viz-cjs 211.14 KB (+0.01% 🔺) 4.3 s (+0.01% 🔺) 3 s (-10.21% 🔽) 7.2 s
polaris-viz-esm 173.45 KB (+0.01% 🔺) 3.5 s (+0.01% 🔺) 2.6 s (-7.31% 🔽) 6.1 s
polaris-viz-css 4.56 KB (0%) 92 ms (0%) 506 ms (+7.29% 🔺) 597 ms
polaris-viz-esnext 178.55 KB (+0.01% 🔺) 3.6 s (+0.01% 🔺) 2.7 s (-21.32% 🔽) 6.2 s

Copy link

@JeremyLudwigDev JeremyLudwigDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎩 Looks good!

@envex envex force-pushed the envex/zero-to-data branch from 60fcb96 to 55c0f71 Compare September 21, 2023 19:28
@Shopify Shopify deleted a comment from github-actions bot Sep 21, 2023
@envex envex force-pushed the envex/zero-to-data branch from 55c0f71 to b8f1ae7 Compare September 21, 2023 19:51
@envex envex force-pushed the envex/zero-to-data branch 2 times, most recently from d64e414 to ddfac56 Compare September 25, 2023 14:55
@envex envex force-pushed the envex/zero-to-data branch from ddfac56 to b7b2090 Compare September 25, 2023 15:01
@envex envex merged commit d830228 into main Sep 25, 2023
6 checks passed
@shopify-shipit shopify-shipit bot temporarily deployed to production September 26, 2023 18:43 Inactive
@envex envex deleted the envex/zero-to-data branch September 12, 2024 18:45
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.

2 participants