-
Notifications
You must be signed in to change notification settings - Fork 114
Update chart-accessibility.mdx #5919
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
base: main
Are you sure you want to change the base?
Conversation
Changed the body text to be more holistic for accessibility
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@karlgoldstraw changes look good lets see if the team has any feedback. |
site/docs/foundations/data-visualization/chart-accessibility.mdx
Outdated
Show resolved
Hide resolved
site/docs/foundations/data-visualization/chart-accessibility.mdx
Outdated
Show resolved
Hide resolved
site/docs/foundations/data-visualization/chart-accessibility.mdx
Outdated
Show resolved
Hide resolved
site/docs/foundations/data-visualization/chart-accessibility.mdx
Outdated
Show resolved
Hide resolved
site/docs/foundations/data-visualization/chart-accessibility.mdx
Outdated
Show resolved
Hide resolved
site/docs/foundations/data-visualization/chart-accessibility.mdx
Outdated
Show resolved
Hide resolved
site/docs/foundations/data-visualization/chart-accessibility.mdx
Outdated
Show resolved
Hide resolved
site/docs/foundations/data-visualization/chart-accessibility.mdx
Outdated
Show resolved
Hide resolved
site/docs/foundations/data-visualization/chart-accessibility.mdx
Outdated
Show resolved
Hide resolved
site/docs/foundations/data-visualization/chart-accessibility.mdx
Outdated
Show resolved
Hide resolved
|
@karlgoldstraw When you commit the new changes, could you convert this PR from Draft to Open so we can request reviews for merging? |
Co-authored-by: Amrita Desmet <[email protected]>
Co-authored-by: Amrita Desmet <[email protected]>
Co-authored-by: Amrita Desmet <[email protected]>
Co-authored-by: Amrita Desmet <[email protected]>
Co-authored-by: Amrita Desmet <[email protected]>
Co-authored-by: Amrita Desmet <[email protected]>
Co-authored-by: Amrita Desmet <[email protected]>
Co-authored-by: Amrita Desmet <[email protected]>
Co-authored-by: Amrita Desmet <[email protected]>
Co-authored-by: Amrita Desmet <[email protected]>
Co-authored-by: Amrita Desmet <[email protected]>
|
Apologies for being pedantic as I just noticed this - could you update the bullet points to start with a capital letter after each semicolon please? The same formatting is used in the other pages. @karlgoldstraw |
Ran yarn prettier
Updated some spellings
Changed the body text to be more holistic for accessibility