Skip to content
This repository has been archived by the owner on Jan 30, 2024. It is now read-only.

bogue: Utilisation du tag <article> dans les composants #373

Open
SuperVignola opened this issue Jul 17, 2020 · 6 comments
Open

bogue: Utilisation du tag <article> dans les composants #373

SuperVignola opened this issue Jul 17, 2020 · 6 comments

Comments

@SuperVignola
Copy link

SuperVignola commented Jul 17, 2020

Description du bogue / Describe the bug
Certains composants modUL utilisent le tag article même si le contenu ne représente pas un « article ». The article role denotes a section of a document, page, or site that, if it were standing on its own, could be viewed as a complete document, page or site (https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/Article_Role).

Voici la liste des composants utilisant le tag article :
- accordion
- accordion-group
- dialog
- message-page
- modal
- panel
- popper
- sitebar
- card

@chuckmah
Copy link
Contributor

Je suis d'accord c'est surement une mauvaise utilisation du tag article dans MODUL

@jipigi @raphpare des idées ?

@raphpare
Copy link
Member

raphpare commented Jul 21, 2020

L'utilisation de la balise article est bien utilisée pour les composants suivants:

  • m-accordion
  • m-dialog
  • m-modal
  • m-sidebar
  • m-panel
  • m-message-page

Le contenu de ces composant, s'il était autonome, pourrait être considéré comme un document, une page ou un site complet.
https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/Article_Role

@SuperVignola
Copy link
Author

@raphpare : je l'ai d'abord remarqué dans la table des matières d'un site de formation dans Brio. La table des matières, c'est le m-sidebar? Il me semble que le contenu de la table des matières ne correspond pas vraiment à la description du tag article.

@jipigi
Copy link
Contributor

jipigi commented Jul 21, 2020

Mon avis là dessus est qu'il est difficile dans MODUL de prévoir quel type de contenu sera placé dans les composants qu'on propose, et donc l'emploi des balises sémantiques de ce type est délicat. Dans certains cas, la balise va être pertinente mais dans d'autres probablement pas. Je pense que je laisserais le soin aux projets de gérer ça. C'est du cas par cas, mais pour la balise article, c'est ma recommandation.

Techniquement, il y a deux attributs (aria-posinset et aria-setsize) qui peuvent compléter la définition de a balise, est-ce qu'il est possible de les définir si la balise est déjà placée à l'intérieur du composant?

@raphpare
Copy link
Member

raphpare commented Jul 21, 2020

Pour les composants dialog, overlay, modal et mesage-page, la balise devrait toujours être article, car ils possèdent tous un titre (entête), un corps et parfois un pied de page. Leur contenu est également toujours indépendant du contenu du site et il peu vivre seul, car ces composants apparaissent dans une fenêtre.

La balise article doit être défini dans ces composants, car elle doit englober l'entête, le corps et le pied de page. Pour les autres composants, la balise article pourrait être retirée.

@jipigi, il serait possible d'ajouter des props pour les attribut aria-posinset et aria-setsize sur les composants en question.

@jipigi
Copy link
Contributor

jipigi commented Jul 21, 2020

@raphpare, non ce n'est pas nécessaire d'ajouter ces propriétés pour le moment, merci.

Pour les composants pour lesquels tu souhaites conserver la balise, est-ce que tu sais quel impact cela aura sur l'expérience des utilisateurs de lecteur d'écran? Est-ce que ça leur sera utile? Si oui, gardons la balise!

Je n'ai personnellement jamais testé le comportement, mais ce que je peux en lire, c'est qu'elle est surtout utilisée pour les flux comme les blog, les nouvelles, les forums, etc.
Examples of appropriate uses of the role="article", or preferably <article>, include blog posts, forum posts, a comment to a forum or blog post, a comment to a comment to a forum or blog post, any an item in a social media feed.

Il y a aussi Deque qui en parle un peu ici dans le cas d'un fil de nouvelles : https://www.deque.com/blog/introduction-to-feed-role-attribute/

Mais on dirait que ça ne s'applique pas vraiment aux fenêtres secondaires. Qu'en penses-tu?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants