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.

Sign up for GitHub

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

Jump to bottom

feat: update FR (french) translation file for next release #1089

Merged
merged 8 commits into from
Aug 27, 2018
Merged

feat: update FR (french) translation file for next release #1089

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
75d641f
feat: update FR (french) translation file for next release
ffoodd Aug 22, 2018
96db21e
fix: forgotten string in FR translation update
ffoodd Aug 22, 2018
39433cb
Merge branch 'develop' into develop
dylanb Aug 22, 2018
64d1e1c
feat(aria-role): Remove non-existing "text" role (#1069)
WilcoFiers Aug 22, 2018
ef8e563
Merge branch 'develop' into develop
dylanb Aug 22, 2018
637550e
fix: typos and plurals after @kloh-fr and @PhilippeVay's proofreading
ffoodd Aug 23, 2018
a99ed10
fix: duplicated word in aria-allowed-role FR translation
ffoodd Aug 27, 2018
722deba
Merge branch 'develop' into develop
WilcoFiers Aug 27, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 0 additions & 7 deletions lib/commons/aria/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1901,13 +1901,6 @@ lookupTable.role = {
implicit: ['dt'],
unsupported: false
},
text: {
type: 'structure',
owned: null,
nameFrom: ['author', 'contents'],
context: null,
unsupported: false
},
textbox: {
type: 'widget',
attributes: {
Expand Down
39 changes: 38 additions & 1 deletion locales/fr.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,10 @@
"description": "S’assure que les attributs ARIA sont autorisés pour le rôle d’un élément",
"help": "Les éléments doivent seulement utiliser les attributs ARIA autorisés"
},
"aria-allowed-role": {
"description": "S’assure que l’attribut role a une valeur valide pour cet élément",
"help": "Le rôle ARIA doit être valide pour cet élément"
},
"aria-dpub-role-fallback": {
"description": "S’assure que les rôles DPUB non supportés ne sont utilisés que sur des éléments avec des rôles implicites",
"help": "Les rôles DPUB non supportés doivent être utilisés sur des éléments avec des rôles implicites"
Expand Down Expand Up @@ -49,6 +53,10 @@
"description": "S’assure que les éléments <audio> ont des sous-titres",
"help": "Les éléments <audio> doivent avoir une psite de sous-titres"
},
"autocomplete-valid": {
"description": "S’assure que l’attribut autocomplete est correct et adapté pour le champ de formulaire",
"help": "Lgattribut autocomplete doit être utilisé correctement"
Copy link

Choose a reason for hiding this comment

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

Typo on the first word (should be L'attribut)

Copy link

Choose a reason for hiding this comment

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

L’attribut even :-)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

French apostrophe ’ is done with Alt Gr + g on Ubuntu with French keyboard… explaining the g x)

},
"blink": {
"description": "S’assure que l’élément <blink> n’est pas utilisé",
"help": "L’élément <blink> est déprécié et ne doit pas être utilisé"
Expand Down Expand Up @@ -121,6 +129,10 @@
"description": "S’assure que l’attribut lang sur l’élément HTML a une valeur valide",
"help": "L’élément <html> doit avoir une valeur valide pour l’attribut lang"
},
"html-xml-lang-mismatch": {
"description": "S’assure que les éléments HTML avec les attributs lang et xml:lang valides indiquent la même langue de base pour la page",
"help": "Les éléments HTML avec les attributs lang et xml:lang doivent avoir la même langue de base"
},
"image-alt": {
"description": "S’assure que les éléments <img> ont un alternative textuelle, ou un rôle none ou presentation",
"help": "Les images doivent avoir un alternative textuelle"
Expand Down Expand Up @@ -279,6 +291,10 @@
"pass": "Les attributs ARIA sont utilisés correctement pour le rôle défini",
"fail": "{{=it.data && it.data.length > 1 ? 'Attributs ARIA non autorisés' : 'Attribut ARIA non autorisé'}} : {{~it.data:value}} {{=value}}{{~}}"
},
"aria-allowed-role": {
"pass": "Le rôle ARIA role est autorisé pour l’élément donné",
Copy link

@PhilippeVay PhilippeVay Aug 24, 2018 •
edited
Loading

Choose a reason for hiding this comment

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

"role" appears twice: a translation of Ensures role attribute has an appropriate value for the element could use the word attribute for one of them.
L'attribut ARIA role (…) for example, edit: or shorter and closer to original sentence L'attribut role.

"fail": "{{=it.data && it.data.length > 1 ? 'Les rôles' : 'Le rôle'}} {{=it.data.join(', ')}} {{=it.data && it.data.length > 1 ? 'ne sont' : 'n’est'}} pas autorisé pour l’élément donné"
Copy link

Choose a reason for hiding this comment

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

Conjugation to correct:
{{=it.data && it.data.length > 1 ? 'ne sont pas autorisés' : 'n’est pas autorisé'}}
instead of:
{{=it.data && it.data.length > 1 ? 'ne sont' : 'n’est'}} pas autorisé

},
"aria-hidden-body": {
"pass": "Aucun attribut aria-hidden n’est présent sur body",
"fail": "aria-hidden=true ne devrait pas être présent sur body"
Expand All @@ -305,12 +321,17 @@
},
"aria-required-children": {
"pass": "Les descendants ARIA requis sont présents",
"fail": "{{=it.data && it.data.length > 1 ? 'Descendants ARIA requis absents' : 'Descendant ARIA requis absent'}} : {{~it.data:value}} {{=value}}{{~}}"
"fail": "{{=it.data && it.data.length > 1 ? 'Descendants ARIA requis absents' : 'Descendant ARIA requis absent'}} : {{~it.data:value}} {{=value}}{{~}}",
"incomplete": "{{=it.data && it.data.length > 1 ? 'Les descendants' : 'Le descendant'}} requièrent le rôle ARIA : {{~it.data:value}} {{=value}}{{~}}"
Copy link

Choose a reason for hiding this comment

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

requièrent is correct with plural but not otherwise.
'Les descendants' : 'Le descendant'}} requièrent should become something like:
'Les descendants requièrent' : 'Le descendant requiert'}}

},
"aria-required-parent": {
"pass": "Les rôles parents ARIA requis sont présents",
"fail": "{{=it.data && it.data.length > 1 ? 'Rôles parents ARIA requis absents' : 'Rôle parent ARIA requis absent'}} : {{~it.data:value}} {{=value}}{{~}}"
},
"unsupportedrole": {
"pass": "Le rôle ARIA est supporté",
"fail": "Le rôle utilisé n’est pas suffisamment supporté par les technologies d’assistance"
},
"aria-valid-attr-value": {
"pass": "Les valeurs d’attribut ARIA sont valides",
"fail": "{{=it.data && it.data.length > 1 ? 'Valeurs d’attribut ARIA invalides' : 'Valeur d’attribut ARIA invalide'}} : {{~it.data:value}} {{=value}}{{~}}"
Expand Down Expand Up @@ -351,6 +372,14 @@
"default": "Impossible de déterminer le rapport de contraste"
}
},
"autocomplete-appropriate": {
"pass": "La valeur autocomplete est sur un élément approprié",
"fail": "La valeur autocomplete est inappropriée sur ce type de champ de formulaire"
},
"autocomplete-valid": {
"pass": "L’attribut autocomplete est formaté correctement",
"fail": "L’attribut autocomplete est formaté incorrectement"
},
"fieldset": {
"pass": "L’élément est contenu dans un fieldset",
"fail": "{{var code = it.data && it.data.failureCode;}}{{? code === 'no-legend'}}Le fieldset n’a pas de legend en premier descendant direct{{?? code === 'empty-legend'}}L’élément legend n’a pas de texte discernable par les lecteurs d’écran{{?? code === 'mixed-inputs'}}Le fieldset contient des champs non liés{{?? code === 'no-group-label'}}Le groupe ARIA n’a pas d’attribut aria-label ou aria-labelledby{{?? code === 'group-mixed-inputs'}}Le groupe ARIA contient des champs non liés {{??}}L’élément n’est contenu ni dans un fieldset ni dans un groupe ARIA{{?}}"
Expand Down Expand Up @@ -407,6 +436,10 @@
"pass": "L’aide à la saisie (title ou aria-describedby) ne duplique pas le contenu du label",
"fail": "L’aide à la saisie (title ou aria-describedby) est identique au contenu du label"
},
"hidden-explicit-label": {
"pass": "L’élément de formulaire a un <label> visible et explicite",
"fail": "L’élément de formulaire a un <label> explicite qui est masqué"
},
"implicit-label": {
"pass": "L’élément de formulaire a un <label> implicite (imbriqué)",
"fail": "L’élément de formulaire n’a pas de <label> implicite (imbriqué)"
Expand All @@ -427,6 +460,10 @@
"pass": "La valeur de l’attribut lang fait partie des codes de langues valides",
"fail": "La valeur de l’attribut lang ne fait pas partie des codes de langues valides"
},
"xml-lang-mismatch": {
"pass": "Les attributs lang et xml:lang indiquent la même langue de base",
"fail": "Les attributs lang et xml:lang indiquent des langues de base différentes"
},
"dlitem": {
"pass": "L’item de liste de description a un élément <dl> parent",
"fail": "L’item de liste de description n’a pas d’élément <dl> parent"
Expand Down
4 changes: 2 additions & 2 deletions test/integration/rules/aria-roles/aria-roles.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,6 @@
<div role="none" id="pass65">ok</div>
<div role="cell" id="pass66">ok</div>
<div role="searchbox" id="pass67">ok</div>
<div role="text" id="pass68">ok</div>
<div role="table" id="pass69">ok</div>
<div role="doc-abstract" id="pass70">ok</div>
<div role="doc-acknowledgments" id="pass71">ok</div>
Expand Down Expand Up @@ -125,6 +124,7 @@
<div role="window" id="fail12">fail</div>
<!-- invalid roles -->
<div role="lol" id="fail13">fail</div>
<div role="text" id="fail14">ok</div>
<!-- unsupported roles -->
<div role="figure" id="fail14">fail</div>
<div role="figure" id="fail15">fail</div>
</div>
5 changes: 3 additions & 2 deletions test/integration/rules/aria-roles/aria-roles.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@
"rule": "aria-roles",
"violations": [
["#fail1"], ["#fail2"], ["#fail3"], ["#fail4"], ["#fail5"], ["#fail6"], ["#fail7"],
["#fail8"], ["#fail9"], ["#fail10"], ["#fail11"], ["#fail12"], ["#fail13"], ["#fail14"]
["#fail8"], ["#fail9"], ["#fail10"], ["#fail11"], ["#fail12"], ["#fail13"], ["#fail14"],
["#fail15"]
],
"passes": [
["#pass1"], ["#pass2"], ["#pass3"], ["#pass4"], ["#pass5"], ["#pass6"], ["#pass7"],
Expand All @@ -15,7 +16,7 @@
["#pass43"], ["#pass44"], ["#pass45"], ["#pass46"], ["#pass47"], ["#pass48"], ["#pass49"],
["#pass50"], ["#pass51"], ["#pass52"], ["#pass53"], ["#pass54"], ["#pass55"], ["#pass56"],
["#pass57"], ["#pass58"], ["#pass59"], ["#pass60"], ["#pass61"], ["#pass62"], ["#pass63"],
["#pass64"], ["#pass65"], ["#pass66"], ["#pass67"], ["#pass68"], ["#pass69"], ["#pass70"],
["#pass64"], ["#pass65"], ["#pass66"], ["#pass67"], ["#pass69"], ["#pass70"],
["#pass71"], ["#pass72"], ["#pass73"], ["#pass74"], ["#pass75"], ["#pass76"], ["#pass77"],
["#pass78"], ["#pass79"], ["#pass80"], ["#pass81"], ["#pass82"], ["#pass83"], ["#pass84"],
["#pass85"], ["#pass86"], ["#pass87"], ["#pass88"], ["#pass89"], ["#pass90"], ["#pass91"],
Expand Down