Skip to content

AccessibilityProject - Linda L #44

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

llindallarsson
Copy link

@NiklasAbrahamsson NiklasAbrahamsson self-requested a review April 8, 2025 07:37
Copy link
Collaborator

@NiklasAbrahamsson NiklasAbrahamsson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Great implementation of the skip button — it improves usability for keyboard and screen reader users.
  • Nicely done highlighting the currently active page, which helps with orientation and navigation.
  • The progress bar is well-executed and informative.
  • Tab order is logical and intuitive, enhancing keyboard navigation.
  • Excellent that initial focus is automatically set to the first radio button — this is an important detail for accessibility.
  • Error handling is implemented effectively, providing helpful and clear feedback to users.

On the results page:

  • Feedback is well-presented and accessible.
  • The Restart Quiz function is nicely integrated and easy to find.
  • Great consideration for screen reader users through the use of accessible icons.

Lastly, it’s good to see that the team has recognized the remaining issues flagged by the WAVE accessibility tool. Acknowledging these areas for improvement shows awareness and a commitment to refinement.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants