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

Accessibility #54

Merged
merged 10 commits into from
Mar 9, 2019
Merged

Accessibility #54

merged 10 commits into from
Mar 9, 2019

Conversation

linsolas
Copy link
Contributor

Hello,

Voici ma traduction pour la page sur l'accessibilité.

@tdd tdd added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Feb 19, 2019
Copy link
Collaborator

@sheplu sheplu left a comment

Choose a reason for hiding this comment

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

une bonne traduction, surtout avec une taille aussi importante !
juste quelques comment qui ont ete transforme en comme


### WCAG {#wcag}

The [Web Content Accessibility Guidelines](https://www.w3.org/WAI/intro/wcag) provides guidelines for creating accessible web sites.
Les [directives pour l'accessibilité au contenu web (*Web Content Accessibility Guidelines*)](https://www.w3.org/WAI/intro/wcag) proposent les lignes directrices pour créer des sites web accessibles.
Copy link
Collaborator

Choose a reason for hiding this comment

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

peut etre preciser l'acronyme ici WCAG puisqu'on s'en sert a la ligne apres


The following resources show us how to do this:
Les ressources ci-dessous nous montrent comme procéder :
Copy link
Collaborator

Choose a reason for hiding this comment

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

comme procéder -> comment procéder

- [The W3C shows us how to label elements](https://www.w3.org/WAI/tutorials/forms/labels/)
- [WebAIM shows us how to label elements](http://webaim.org/techniques/forms/controls)
- [The Paciello Group explains accessible names](https://www.paciellogroup.com/blog/2017/04/what-is-an-accessible-name/)
- [Le W3C nous montre comme étiqueter les éléments](https://www.w3.org/WAI/tutorials/forms/labels/).
Copy link
Collaborator

Choose a reason for hiding this comment

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

comme étiqueter -> comment étiqueter

- [WebAIM shows us how to label elements](http://webaim.org/techniques/forms/controls)
- [The Paciello Group explains accessible names](https://www.paciellogroup.com/blog/2017/04/what-is-an-accessible-name/)
- [Le W3C nous montre comme étiqueter les éléments](https://www.w3.org/WAI/tutorials/forms/labels/).
- [Le WebAIM nous montre comme étiqueter les éléments](http://webaim.org/techniques/forms/controls).
Copy link
Collaborator

Choose a reason for hiding this comment

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

comment

content/docs/accessibility.md Outdated Show resolved Hide resolved

This is one example of many cases where depending on only pointer and mouse events will break functionality for keyboard users. Always testing with the keyboard will immediately
highlight the problem areas which can then be fixed by using keyboard aware event handlers.
C'est un exemple des nombreux cas où le fait de ne dépendre que des événements de souris et de pointeur casse les fonctionnalités pour les utilisateurs de clavier. Toujours tester avec le clavier mettra immédiatement en évidence les problèmes qui peuvent ensuite être résolus à l'aide des gestionnaires d'événement clavier.
Copy link
Collaborator

Choose a reason for hiding this comment

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

gestionnaires d'événement clavier. -> gestionnaires d'événements clavier.


- [WebAIM - Color Contrast Checker](http://webaim.org/resources/contrastchecker/)
- [The Paciello Group - Color Contrast Analyzer](https://www.paciellogroup.com/resources/contrastanalyser/)
- [WebAIM - Vérification de contraste de couleur](http://webaim.org/resources/contrastchecker/).
Copy link
Collaborator

Choose a reason for hiding this comment

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

couleurs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

J'ai changé en "contraste des couleurs", au moins le doute n'est plus permis 😅

- [WebAIM - Color Contrast Checker](http://webaim.org/resources/contrastchecker/)
- [The Paciello Group - Color Contrast Analyzer](https://www.paciellogroup.com/resources/contrastanalyser/)
- [WebAIM - Vérification de contraste de couleur](http://webaim.org/resources/contrastchecker/).
- [The Paciello Group - Analyseur de contraste de couleur](https://www.paciellogroup.com/resources/contrastanalyser/).
Copy link
Collaborator

Choose a reason for hiding this comment

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

couleurs


By far the easiest and also one of the most important checks is to test if your entire website can be reached and used with the keyboard alone. Do this by:
De loin la vérification plus simple, mais aussi l'une des plus importantes, consiste à tester si l'ensemble de votre site web est accessible et utilisable avec le clavier seul. Procédez ainsi :
Copy link
Collaborator

Choose a reason for hiding this comment

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

la vérification plus simple -> la vérification la plus simple

@linsolas
Copy link
Contributor Author

Voilà, traduction mise à jour.

@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for fr-reactjs ready!

Built with commit 1e235be

https://deploy-preview-54--fr-reactjs.netlify.com

@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for fr-reactjs ready!

Built with commit 7637998

https://deploy-preview-54--fr-reactjs.netlify.com

Copy link
Collaborator

@tdd tdd left a comment

Choose a reason for hiding this comment

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

Salut @linsolas !

Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.

Je devrais avoir le temps ce jour de faire plusieurs revues sur tes PR restantes… C'est comme toujours du très bon boulot, comme en témoigne le fait que pour > 500 lignes je n'ai « que » 63 modifs, la plupart étant regroupées sur le même tweak (genre “widget” -> « élément d’interface » et “support” -> « prise en charge »).

Je compte sur toi pour y jeter un coup d'œil depuis tes vacances, comme promis 😉

Tout est fait sous forme de Suggestions GitHub, il te suffit à chaque suggestion que tu approuves de cliquer le bouton d'ajout au lot de modifs à committer, et sur la fin de créer le commit groupé. (ce mode de regroupement en un seul commit est à favoriser absolument, mais tu devras aller dans l'onglet Files changed pour y accéder, il n'est pas utilisable dans l’onglet Conversation)

Si certaines appellent discussion, n'hésite pas, tu réponds, on en parle, on avance ensemble 🤝

Quand tout sera bouclé côté code, je ferai une repasse visuelle finale en local avant de fusionner.

Encore mille mercis, j'ai hâte de voir cette PR finalisée et dans master !

content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
@tdd tdd added Reviewed and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Mar 8, 2019
@tdd tdd self-assigned this Mar 8, 2019
Co-Authored-By: linsolas <linsolas@gmail.com>
Copy link
Contributor Author

@linsolas linsolas left a comment

Choose a reason for hiding this comment

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

Merci pour ta relecture @tdd.
Dis-moi s'il faut ajouter le (en anglais) après les liens anglophones ?
Sinon j'ai laissé une suggestion de côté. À voir...

content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
@tdd
Copy link
Collaborator

tdd commented Mar 9, 2019

@linsolas tu peux ajouter « (en anglais) » seulement après les liens qui sont dans les listes de ressources (pour les liens en plein cœur de phrase, à toi de juger au cas par cas si ça rompt trop la phrase ou non)

@linsolas
Copy link
Contributor Author

linsolas commented Mar 9, 2019

Hello @tdd,

J'ai mis à jour le texte en ajoutant la plupart du temps le (en anglais) quand ça n'alourdit pas le texte. J'ai trouvé 2 / 3 traductions supplémentaires de sites aussi.

@tdd tdd merged commit 089b252 into reactjs:master Mar 9, 2019
@linsolas linsolas deleted the accessibility branch March 9, 2019 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants