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

Fix/issue git #443

Merged
merged 3 commits into from
Jul 12, 2023
Merged

Fix/issue git #443

merged 3 commits into from
Jul 12, 2023

Conversation

SimonF30
Copy link
Contributor

@netlify
Copy link

netlify bot commented Jul 10, 2023

Deploy Preview for a11y-guidelines-orange ready!

Name Link
🔨 Latest commit f0d790b
🔍 Latest deploy log https://app.netlify.com/sites/a11y-guidelines-orange/deploys/64ae760ec1dcb10008aad0ab
😎 Deploy Preview https://deploy-preview-443--a11y-guidelines-orange.netlify.app/fr/web/developper/couleurs-et-contrastes
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

- Cas particulier des liens non soulignés :
- Ces liens doivent avoir un rapport de contraste de 3:1 avec le texte qui les entoure
- Ces liens doivent avoir un autre moyen que la couleur permettant de les distinguer au focus clavier ou au survol souris
- S'assurer que Les objets graphiques porteurs d'information non-textuelle, les composants d’interface utilisateur et les états de l'interface (focus, hover...) ont un contraste de 3:1. Sont concernés, entre autres : les boutons, les boutons radios, les cases à cocher, les listes de sélection, les menus et volets de navigation, les barres d’outils, les onglets, les carrousels, les curseurs, les barres de progression, les bulles d’aides, les barres de progression, les graphiques…
- S'assurer que es objets graphiques porteurs d'information non-textuelle, les composants d’interface utilisateur et les états de l'interface (focus, hover...) ont un contraste de 3:1. Sont concernés, entre autres : les boutons, les boutons radios, les cases à cocher, les listes de sélection, les menus et volets de navigation, les barres d’outils, les onglets, les carrousels, les curseurs, les barres de progression, les bulles d’aides, les barres de progression, les graphiques…
Copy link
Member

@isabellechanclou isabellechanclou Jul 12, 2023

Choose a reason for hiding this comment

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

Remplace "es" by "les" in "S'assurer que es objets graphiques..."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oupsi


On n’est pas tenus dappliquer ce critère :
Nous ne sommes pas tenus d'appliquer ce critère :
Copy link
Member

Choose a reason for hiding this comment

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

Why this change of apostrophe in "d'appliquer" ?


On n’est pas tenus dappliquer ce critère :
Nous ne sommes pas tenus d'appliquer ce critère :
Copy link
Member

Choose a reason for hiding this comment

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

Why this change of apostrophe in "d'appliquer" ?

Copy link
Member

@isabellechanclou isabellechanclou left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@SimonF30 SimonF30 merged commit cef1fbd into master Jul 12, 2023
5 checks passed
@SimonF30 SimonF30 deleted the fix/issue-git branch July 12, 2023 13:49
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.

None yet

2 participants