-
Notifications
You must be signed in to change notification settings - Fork 27
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
Limit Donut legend to 50% of chart width #1646
Conversation
size-limit report 📦
|
d60e7b6
to
8dff0e1
Compare
f0a4b1b
to
efc72ef
Compare
@michaelnesen Can you look over this when you have some time, you have the most context around |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! 👍
donut.mp4
One comment about the table spacing 👇
@@ -2,4 +2,5 @@ | |||
width: 100%; | |||
border-collapse: separate; | |||
border-spacing: 0 10px; | |||
table-layout: fixed; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With table-layout: fixed
i'm seeing some extra spacing in the table. It's only really visible with the legend positioned to the left as it pushes the viz to the right
Perhaps adding a width
to the trend indictor table cell could fix this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, fixed
makes all the cells fill the space unless they have an explicit width. I'll add it!
efc72ef
to
c3a5eac
Compare
c3a5eac
to
e371426
Compare
What does this implement/fix?
Limit the
DonutChart
legends from taking up more than 50% of the chart space.Does this close any currently open issues?
https://github.com/Shopify/core-issues/issues/68612
What do the changes look like?
Storybook link
https://6062ad4a2d14cd0021539c1b-xydvmesepk.chromatic.com/?path=/story/polaris-viz-charts-donutchart--truncated-legends
Before merging
Check your changes on a variety of browsers and devices.
Update the Changelog's Unreleased section with your changes.
Update relevant documentation, tests, and Storybook.
Make sure you're exporting any new shared Components, Types and Utilities from the top level index file of the package