-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
[docs][joy-ui] Add use-count-up
integration with the Circular Progress
#38952
[docs][joy-ui] Add use-count-up
integration with the Circular Progress
#38952
Conversation
<!-- Thanks so much for your PR, your contribution is appreciated! ❤️ --> - [x] I have followed (at least) the [PR section of the contributing guide](https://github.com/mui/material-ui/blob/HEAD/CONTRIBUTING.md#sending-a-pull-request).
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.
Netlify deploy previewBundle size report |
use-count-up
intergration with the Circular Progress
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.
Sweet, thanks for persisting with this! Looks great to me!
Would still wait for either @siriwatknp's or @DiegoAndai's (given you were seeing some flaky test issues with this integration) perspective before merging, though. :)
We will skip the Argos test to avoid the flaky test from the Linear Progress example: #38969. We should do the same here. It's a new snapshot, so I don't know if that will be a problem, but let's try 😅 |
…ular-progress-count-up-feature-v2
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.
👏 Thanks again @anon-phantom
use-count-up
intergration with the Circular Progressuse-count-up
integration with the Circular Progress
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.
I fixed some type issues and polished the demo.
Closes #38682
https://deploy-preview-38952--material-ui.netlify.app/joy-ui/react-circular-progress/#use-count-up