feat: affichage de la devise configurable #10

Merged
tcharlss merged 3 commits from issue_9_affichage_devise into master 5 months ago
Owner

Nouvelle option de configuration pour choisir si on veut afficher le code ou le symbole de la devise dans les montants.

Refs: #9

edit : je voulais pas faire des labels trop verbeux, mais pour être clair l'option ne concerne que l'affichage dans les montants (avec montant_formater). Faudrait peut-être quand même y caser ce terme ?

Nouvelle option de configuration pour choisir si on veut afficher le code ou le symbole de la devise dans les montants. Refs: #9 ![](https://git.spip.net/attachments/9ca2c766-7063-4c85-ab16-2b1b77bffc6d) edit : je voulais pas faire des labels trop verbeux, mais pour être clair l'option ne concerne que l'affichage dans les *montants* (avec montant_formater). Faudrait peut-être quand même y caser ce terme ?
tcharlss added 1 commit 5 months ago
5f3b2522f4 feat: affichage de la devise configurable
tcharlss added 1 commit 5 months ago
rastapopoulos reviewed 5 months ago
rastapopoulos left a comment
Owner

Moi ce qui manque je trouve c'est le terme "par défaut", car ensuite on peut toujours faire un appel plus précis avec d'autres options.

Donc "Affichage par défaut de la devise" ?

Sinon pour la génération, ça se contente de changer l'opption envoyée à la lib mais du coup ça n'utilise pas les techniques pour augmenter l'accessibilité quand on n'affiche que le symbole ?

Il me semble qu'on ne peut pas juste présenter un choix "si simple" comme ça comme si ça ne changeait rien, alors que seul avec le code est accessible entièrement et l'autre non. D'autant plus quand on parle de ce qui va se générer par défaut partout dans le site (pas juste sur tel truc ponctuel où c'est moins grave de faire le choix de pas être accessible pour faire joli à un endroit précis)

Moi ce qui manque je trouve c'est le terme "par défaut", car ensuite on peut toujours faire un appel plus précis avec d'autres options. Donc "Affichage par défaut de la devise" ? Sinon pour la génération, ça se contente de changer l'opption envoyée à la lib mais du coup ça n'utilise pas les techniques pour augmenter l'accessibilité quand on n'affiche que le symbole ? Il me semble qu'on ne peut pas juste présenter un choix "si simple" comme ça comme si ça ne changeait rien, alors que seul avec le code est accessible entièrement et l'autre non. D'autant plus quand on parle de ce qui va se générer par défaut partout dans le site (pas juste sur tel truc ponctuel où c'est moins grave de faire le choix de pas être accessible pour faire joli à un endroit précis)
Poster
Owner

Nan c'est bon, ça produit rigoureusement le même markup avec la devise/symbole dans un span et les mêmes trucs pour l'accessibilité, ça devrait pas poser de problème de ce point de vue.

Exemple :

<span class="montant" data-montant-nombre="100" data-montant-devise="EUR">
	100,00&nbsp;<span class="montant__devise"></span>
	<meta itemprop="price" content="100">
	<meta itemprop="priceCurrency" content="EUR">
</span>

Y a que le contenu du span.montant__devise qui change.

Et ok pour le label.

Nan c'est bon, ça produit rigoureusement le même markup avec la devise/symbole dans un span et les mêmes trucs pour l'accessibilité, ça devrait pas poser de problème de ce point de vue. Exemple : ```html <span class="montant" data-montant-nombre="100" data-montant-devise="EUR"> 100,00&nbsp;<span class="montant__devise">€</span> <meta itemprop="price" content="100"> <meta itemprop="priceCurrency" content="EUR"> </span> ``` Y a que le contenu du `span.montant__devise` qui change. Et ok pour le label.

dans un span et les mêmes trucs pour l'accessibilité

justement,càd ? :p

dans le code que tu recolles là, il n'y a rigoureusement rien d'accessibilité, est-ce qu'il n'y a pas une confusion entre microdata (qui est pour Google, les robots etc, comprendre informatiquement qu'il y a des produits avec tels prix dans une page), et l'accessibilité ?

ce n'est pas à ça que je pensais quand j'évoquais les techniques d'accessibilité pour les symboles de devise, il me semblait avoir vu passer des techniques avec des aria pour pas lire le contenu, et que le vrai code soit dans une balise hidden à l'œil mais lu en screen reader, etc (faut que je retrouve où)

> dans un span et les mêmes trucs pour l'accessibilité justement,càd ? :p dans le code que tu recolles là, il n'y a rigoureusement rien d'accessibilité, est-ce qu'il n'y a pas une confusion entre microdata (qui est pour Google, les robots etc, comprendre informatiquement qu'il y a des produits avec tels prix dans une page), et l'accessibilité ? ce n'est pas à ça que je pensais quand j'évoquais les techniques d'accessibilité pour les *symboles* de devise, il me semblait avoir vu passer des techniques avec des aria pour pas lire le contenu, et que le vrai code soit dans une balise hidden à l'œil mais lu en screen reader, etc (faut que je retrouve où)

Ah bé c'était la discussion dans #4 donc :) (mais j'ai déjà vu un truc en parler ailleurs mais impossible de retrouver)

Ah bé c'était la discussion dans #4 donc :) (mais j'ai déjà vu un truc en parler ailleurs mais impossible de retrouver)
Poster
Owner

Alors oui, en fait on n'avait rien mis en plus pour l'accessibilité parcequ'il n'y avait pas besoin à priori. À confirmer mais me semble bien que c'était suffisant comme ça.

Et j'ai pas l'impression que passer du code au symbole change quelque chose de ce point de vue là, de ce que je lis, les trucs de vocalisation savent lire les symboles €, $ et autre.

Dans cet article par exemple, le coup de tout mettre en aria-hidden et de mettre le texte accessible visuellement caché après, c'est que pour les cas spéciaux : https://a11y-guidelines.orange.com/en/web/components-examples/price-vocalization/

Leur exemple :

<p class="example" aria-hidden="true">
À partir de
<span class="price">
    120<sup>.90€</sup><sub>/mois</sub>
</span>
<p>
<span class="visually-hidden">à partir de 120.90€ par mois</span>

Dans la partie réservée aux lecteurs, ils mettent bien un symbole.

Alors oui, en fait on n'avait rien mis *en plus* pour l'accessibilité parcequ'il n'y avait pas besoin à priori. À confirmer mais me semble bien que c'était suffisant comme ça. Et j'ai pas l'impression que passer du code au symbole change quelque chose de ce point de vue là, de ce que je lis, les trucs de vocalisation savent lire les symboles €, $ et autre. Dans cet article par exemple, le coup de tout mettre en aria-hidden et de mettre le texte accessible visuellement caché *après*, c'est que pour les cas spéciaux : https://a11y-guidelines.orange.com/en/web/components-examples/price-vocalization/ Leur exemple : ```html <p class="example" aria-hidden="true"> À partir de <span class="price"> 120<sup>.90€</sup><sub>/mois</sub> </span> <p> <span class="visually-hidden">à partir de 120.90€ par mois</span> ``` Dans la partie réservée aux lecteurs, ils mettent bien un symbole.
Poster
Owner

Ah j'ai posté après avoir lu la réponse :p

Pour être sûrs de chez sûrs on pourrait faire le #4 en même temps ouais, mais c'est just un truc que j'avais noté au passage, je sais pas si c'est vraiment nécessaire dans notre cas. Faudrait confirmation d'un⋅e expert⋅e

Et c'est enquiquinant si ça oblige à charger une css juste pour ça en effet.

Ah j'ai posté après avoir lu la réponse :p Pour être sûrs de chez sûrs on pourrait faire le #4 en même temps ouais, mais c'est just un truc que j'avais noté au passage, je sais pas si c'est vraiment nécessaire dans notre cas. Faudrait confirmation d'un⋅e expert⋅e Et c'est enquiquinant si ça oblige à charger une css juste pour ça en effet.

Ouais faudrait un appel à un joker :p

Je ne sais pas si désormais tous les symboles sont bien accessibles (ça évolue évidemment au fil des années, et peut-être à terme on peut changer le défaut…). Les 4 trucs connus d'accord, $€£¥, mais bon ya… un peu plus de pays et de devises que ça.

Mais peut-être on peut considérer que celles moins connues bah les gens ont qu'à laisser en code par défaut, et faire un markup en différents eux mêmes…

Ouais faudrait un appel à un joker :p Je ne sais pas si désormais tous les symboles sont bien accessibles (ça évolue évidemment au fil des années, et peut-être à terme on peut changer le défaut…). Les 4 trucs connus d'accord, $€£¥, mais bon ya… un peu plus de pays et de devises que ça. Mais peut-être on peut considérer que celles moins connues bah les gens ont qu'à laisser en code par défaut, et faire un markup en différents eux mêmes…
tcharlss added 1 commit 5 months ago
Poster
Owner

À vue de nez sur /?exec=devises&debut_liste_devises=-1#pagination_liste_devises, ça à l'air d'être un peu moins d'1/4 des devises qui ont un symbole.

Ouais je sais pas comment les trucs de vocalisation s'en sortent avec les symboles genre $BZ, R$, ₹ et cie.

Mais bon :

on peut considérer que celles moins connues bah les gens ont qu'à laisser en code par défaut, et faire un markup en différents eux mêmes…

Donc option à utiliser en connaissance de cause, ça me va !

Voilà en ajoutant un fieldset et revoyant le label :

À vue de nez sur `/?exec=devises&debut_liste_devises=-1#pagination_liste_devises`, ça à l'air d'être un peu moins d'1/4 des devises qui ont un symbole. Ouais je sais pas comment les trucs de vocalisation s'en sortent avec les symboles genre $BZ, R$, ₹ et cie. Mais bon : > on peut considérer que celles moins connues bah les gens ont qu'à laisser en code par défaut, et faire un markup en différents eux mêmes… Donc option à utiliser en connaissance de cause, ça me va ! Voilà en ajoutant un fieldset et revoyant le label : ![](https://git.spip.net/attachments/fa8c6330-21c4-478d-b8cd-4ba8ead80b48)

Pour l'instant ok, et comme je disais SI, je dis bien SI, une personne confirme que les symboles sont majoritairement ok en 2022 pour l'accessibilité, on pourra même changer le défaut (donc revenir à du symbole par défaut)

Pour l'instant ok, et comme je disais SI, je dis bien SI, une personne confirme que les symboles sont majoritairement ok en 2022 pour l'accessibilité, on pourra même changer le défaut (donc revenir à du symbole par défaut)
rastapopoulos approved these changes 5 months ago
tcharlss merged commit 69eaceed4f into master 5 months ago
tcharlss deleted branch issue_9_affichage_devise 5 months ago

Reviewers

rastapopoulos approved these changes 5 months ago
The pull request has been merged as 69eaceed4f.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.