-
-
Notifications
You must be signed in to change notification settings - Fork 31.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
[docs] Revise Joy UI "Circular Progress" page #36126
Conversation
|
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 @LadyBluenotes ! This is looking great. One thing the page could still use is an explanation of the value
prop. Maybe this would make sense to cover in the Basics section?
As for details on accessibility, I'd say the more the merrier! You could definitely talk about the aria labels.
docs/data/joy/components/circular-progress/circular-progress.md
Outdated
Show resolved
Hide resolved
docs/data/joy/components/circular-progress/circular-progress.md
Outdated
Show resolved
Hide resolved
docs/data/joy/components/circular-progress/circular-progress.md
Outdated
Show resolved
Hide resolved
docs/data/joy/components/circular-progress/circular-progress.md
Outdated
Show resolved
Hide resolved
docs/data/joy/components/circular-progress/circular-progress.md
Outdated
Show resolved
Hide resolved
docs/data/joy/components/circular-progress/circular-progress.md
Outdated
Show resolved
Hide resolved
docs/data/joy/components/circular-progress/circular-progress.md
Outdated
Show resolved
Hide resolved
docs/data/joy/components/circular-progress/circular-progress.md
Outdated
Show resolved
Hide resolved
docs/data/joy/components/circular-progress/circular-progress.md
Outdated
Show resolved
Hide resolved
Was reaching out to see if there was anything else I needed to update for 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.
👍 thanks for you contribution!
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.
Great job as always @LadyBluenotes !
Part of #33998
Hey @samuelsycamore!
Thought I'd try my hand at another.
When I was doing this one, it didn't have an accessibility component, so I added the color ratio part. I noticed built into the component, there's ARIA labels already. I was contemplating adding them to the accessibility part but I'm not sure whether you're adding how to make components accessible in this, or explaining some of the out-of-the-box things.
The anatomy section also threw me off a bit. I know that adding the classes is important but I wasn't able to find any examples that had something similar to this one, where SVG tags and a circle tag was involved (really didn't even know this was an HTML tag lol).
Let me know what you think and if there's anything else I need to revise!
Hope you're having a great day,