Skip to content

Commit

Permalink
Apply Margaree's suggestions
Browse files Browse the repository at this point in the history
Co-authored-by: Margaree Peacocke <margaree.p@gmail.com>
  • Loading branch information
ChrisLabattD2L and margaree authored Jan 21, 2025
1 parent 9a9944c commit bce0aaf
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions components/expand-collapse/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,8 @@ The `d2l-expand-collapse-content` element can be used to create expandable and c
To make your usage of `d2l-expand-collapse-content` accessible, it should follow the [W3C Disclosure (Show/Hide) Pattern](https://www.w3.org/WAI/ARIA/apg/patterns/disclosure/).

To achieve this, the control that toggles the expanded state should:
- Should use the [`d2l-button`](../button) or use an element with the [`button`](https://w3c.github.io/aria/#button) role
- Use the [`d2l-button`](../button) or use an element with the [`button`](https://w3c.github.io/aria/#button) role
- Toggle between states when using the `Enter` and `Space` buttons and retain focus upon toggle
- Have the [`aria-expanded`](https://www.w3.org/TR/wai-aria/#aria-expanded) attribute, so screen reader users will know what state it's in
- Have the [`aria-expanded`](https://www.w3.org/TR/wai-aria/#aria-expanded) attribute set to `'true'` or `'false'` depending on expansion state so that screen reader users will know what state it's in
- Be adjacent to the expanded/collapsed content

0 comments on commit bce0aaf

Please sign in to comment.