Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Button and link a11y docs #924
Button and link a11y docs #924
Changes from all commits
9c9f783
39d1b54
ecce3ec
b9f38f2
1238cfe
ca9e383
016c57d
6ccae15
85d1c9a
0eb1e78
5da2eae
48f099f
af046ce
e4bf84c
ca42994
be04b96
de37a49
110e6dd
5d9fab9
e08834b
b59337c
94210c3
f4c5bde
d7beb60
2308160
1e04789
3e52023
f61415b
2690df4
25feebc
6375184
ee89b26
3670720
c60a031
2779012
cda1178
c20f01b
a664e02
d27cdfe
3b9f974
ebe8ddb
023c599
ece28ed
f5ff638
f0c7e43
1b536bb
5a70def
6ce31b7
b602322
e4f56cd
b75ec5c
39c31e2
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
iOS and Android content for the button page, since there are separate tabs. We don't currently have native-specific guidance so I've linked to the platform best practices.
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.
Although it's a bit awkward from the "Web" perspective to begin the section with external resources that don't apply to web, my understanding is that this will eventually change, when we have native-specific guidance.
To make it more clear what content applies to the reader:
We could add an intro sentence under the Accessibility heading that lets readers know that the accessibility guidelines for the button component differ for Web, Android, and iOS. This sets the reader up to expect to look for the content that applies to them.
Perhaps rearrange the leading sentences to start with "For Android, see Material Design ... " and "For iOS, see Apple's ..."
Perhaps add Android, iOS, and Web headings
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.
@sadiesaurus The sections here display on the web, iOS, and Android tabs in the UI in the style guide, which is what the
<! content-for: ...>
tags are doing in the file. The Mobile-specific example screenshot is showing what would show up for Android on that tab, while the other specific content shows up for each tab specified. That's what the first item in the 🎩 recommendations is doing, pulling from the PR that @kaelig worked on to do this formatting. Let me know if that's clear!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.
Ohhhh, yes now I see. I didn't 🎩 it properly last time. This works well!
Please disregard my comment! 😅
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 agree that headings would help with the Markdown readability in GitHub itself. I couldn't find a solid and elegant way to make it work :/
@BPScott and I might have cycles to look into it later this year.