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

Tiny accessibility fixes #10

Merged
merged 1 commit into from
Mar 20, 2024
Merged

Tiny accessibility fixes #10

merged 1 commit into from
Mar 20, 2024

Conversation

manuhabitela
Copy link
Contributor

@manuhabitela manuhabitela commented Mar 20, 2024

👋

Those are micro changes to better match our accessibility standards:

  • add the website context in the title
  • change link color to match color contrast ratio required by RGAA and underline links to mark the difference with surrounding text. I chose DSFR blue
  • add an alt text for the login button, for screen readers
  • wrap everything in a <main> tag as nothing should be outside region tags in theory

I know it's not that necessary on those pages but now they can pass automated a11y tests if we setup any (see numerique-gouv/moncomptepro#508) :)

@rdubigny @douglasduteil

Those are micro changes to better match our accessibility standards:

- add the website context in the title
- change link color to match color contrast ratio required by RGAA and
underline links to mark the difference with surrounding text. Chose DSFR
blue
- wrap everything in a <main> tag as nothing should be outside region
tags in theory

I know it's not that necessary on those pages but now they can pass
automated a11y tests if we setup any :)
@rdubigny rdubigny merged commit 98cbee8 into numerique-gouv:master Mar 20, 2024
3 checks passed
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.

2 participants