-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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(Loading): increase stroke-dashoffset #7499
fix(Loading): increase stroke-dashoffset #7499
Conversation
✔️ Deploy preview for carbon-elements ready! 🔨 Explore the source changes: 2cbd8df 🔍 Inspect the deploy logs: https://app.netlify.com/sites/carbon-elements/deploys/6007659e0685d800076f0cb6 😎 Browse the preview: https://deploy-preview-7499--carbon-elements.netlify.app |
✔️ Deploy preview for carbon-components-react ready! 🔨 Explore the source changes: 2cbd8df 🔍 Inspect the deploy logs: https://app.netlify.com/sites/carbon-components-react/deploys/6007659edc06b70007522072 😎 Browse the preview: https://deploy-preview-7499--carbon-components-react.netlify.app |
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.
@emyarod looks good 🍔 !
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 good!
Closes #7497
This PR adjusts the small loading spinner
stroke-dashoffset
values according to audit feedback. Not sure if the suggested value achieves the desired effect though (requires visual review), since the referenced style rules were different from the example screenshots