Skip to content

Accessibility improvements#1078

Merged
ArtOfCode- merged 22 commits intocodidact:developfrom
eip-ewi:accessibility-improvements
Jun 21, 2023
Merged

Accessibility improvements#1078
ArtOfCode- merged 22 commits intocodidact:developfrom
eip-ewi:accessibility-improvements

Conversation

@Taeir
Copy link
Contributor

@Taeir Taeir commented Jun 19, 2023

This PR contains multiple improvements for accessibility, screen readers, voice control and reader views. It may also improve SEO.

Note that this probably does not fix all accessibility issues, but it should at least help for the most-used pages. I expect a few quite small changes have a big impact, such as marking the sidebar as a sidebar.

Main changes:

  • Items which should be buttons (effect on current page) are now also classified as buttons
  • Added aria-labels (screen readers) for many buttons to make them understandable without surrounding visual content.
  • Signal sidebar as sidebar
  • Signal comment threads and comments as lists
  • Signal the search field(s) and forms correctly

Changes also in this PR:

Future recommendations:

  • Buttons with short context-dependent descriptions (e.g. "Show more" or "Unfollow new"), should have longer aria-label descriptions to explain their meaning when the user cannot use visual clues.

Fixes #404
Fixes #1068

Taeir added 6 commits June 19, 2023 21:39
The buttons that immediately take action are now correctly set to role
button, such that screen readers / reader views understand them as such.
The History and Edit button open new pages, so those are considered
actual links.
@Taeir Taeir force-pushed the accessibility-improvements branch from 818efc5 to 9da4b6e Compare June 19, 2023 20:20
@Taeir Taeir requested a review from MoshiKoi June 19, 2023 23:08
@Taeir Taeir marked this pull request as ready for review June 19, 2023 23:09
@ArtOfCode- ArtOfCode- merged commit 6098166 into codidact:develop Jun 21, 2023
@Taeir Taeir deleted the accessibility-improvements branch July 21, 2023 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Screen Readers do not properly parse qpixel pages Answer comment links from activity page don't work (take you to the page but not the comment)

2 participants