Skip to content

Commit

Permalink
prettify
Browse files Browse the repository at this point in the history
  • Loading branch information
davidhunter08 committed Oct 16, 2024
1 parent b1951d2 commit f58d1bd
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion docs/_includes/layouts/icons-example.njk
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@
<tr>
<td>{{ nhsappIcon('sort') }}</td>
<td>sort</td>
<td><a target="_blank" rel="noopener noreferrer" href="https://github.com/nhsuk/nhsapp-frontend/tree/main/src/assets/icons/messages-unread.svg">sort.svg</a></td>
<td><a target="_blank" rel="noopener noreferrer" href="https://github.com/nhsuk/nhsapp-frontend/tree/main/src/assets/icons/messages-unread.svg">sortgit .svg</a></td>
<td></td>
</tr>
<tr>
Expand Down
4 changes: 2 additions & 2 deletions docs/components/icons.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,12 +35,12 @@ You can find the [icons in Figma](https://www.figma.com/design/6f2CbcZ7cnpNrtKEc

## Accessibility

Icons that act as functional buttons should have aria labels, so that they are announced to screen reader users. If an icon is not functional, and it appears alongside supporting text that conveys the same meaning, it may not need alt-text or an aria label.
Icons that act as functional buttons should have aria labels, so that they are announced to screen reader users. If an icon is not functional, and it appears alongside supporting text that conveys the same meaning, it may not need alt-text or an aria label.

Designers and developers should work together to understand the most accessible option in context, considering [NHS Service Manual accessibility guidance](https://service-manual.nhs.uk/accessibility/design).

## Research

The icons used on the top navigation and bottom navigation of the NHS App were well-recognised when shown to users without supporting text in our research sessions.
The icons used on the top navigation and bottom navigation of the NHS App were well-recognised when shown to users without supporting text in our research sessions.

Participants generally understood the meaning of the icons. However, the research supported the need for some icons to have accompanying text to make the meaning clear.

0 comments on commit f58d1bd

Please sign in to comment.