add a11y notes for buttons - #27
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| #### No animation | ||
|
|
||
| <Alert intent="info" title="A11Y Requirement"> | ||
| - Do not use this variant if accessibility is a must have criterion and the component it's used in features multiple buttons in a row or column. |
There was a problem hiding this comment.
polish (non-blocking): Could we get a link or something that explains why this is important? I'm not sure why having multiple buttons in a row would change the requirements :)
There was a problem hiding this comment.
@danbahrami its basically the non text contrast criterion which states that:
This success criterion does not require that changes in color that differentiate between states of an
individual component meet the 3:1 contrast ratio when they do not appear next to each other.
As multiple buttons in a row or column would appear next to each other i'd strongly recommend to make sure we fulfil any eventualities.
| <Alert intent="info" title="A11Y Requirement"> | ||
| - Do not use tertiary buttons as replacements for text links in inline text as they might not provide enough contrast to surrouding text. | ||
| - These buttons are not intended to be used as tab or accordion controls or any other control requiring an active state. | ||
| </Alert> |
There was a problem hiding this comment.
thought (non-blocking): It feels a bit too strong to come right in with this at the top before we even see any examples.
Wondering if we could move this down a little bit?
2 participants
No description provided.