Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Improving Accessibility and Broken Links #7

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

Conversation

qtop1
Copy link

@qtop1 qtop1 commented Jun 9, 2024

Hi,

I ran the website through a few online accessibility checker(EXPERTE, accessibilitychecker, accessibe ) to test how accessible the site is for disabled users.

All sites came back with the following pitfalls :

  • Background and foreground not having a sufficient contrast::: _ I did not alter this element during my contribution as I didn't want to change the colour palette of the site, without discussing. But this something could be considered for the future._

  • Some images elements do not have alt attributes ::: Implemented alt text within contribution

  • Link "" does not have a discernible name ::: Implemented aria-label to give the link a short description which will be accessible to users with a screen reader

  • Disabling the zoom function can make it difficult for visually impaired users to view the site. ::: Implemented this fix, the change the max scale to 5

Also I fixed the broken links within the project.

Please see the accessibility rating before fixing the issues :::
Screenshot 2024-06-09 at 15 41 28
Accessibility Check

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.

1 participant