Produits Entr'ouvert » w.c.s.

Pourquoi ajouter un tag button ? Cela complique tout : le JS, le CSS.
Je suis partisan d'ajouter simplement un attribut `role="button"` au h2

Pourquoi ajouter un tag button ? Cela complique tout : le JS, le CSS.

Ça correspond à ce que note l'audit "le composant qui permet d’activer l’ouverture ou la fermeture des panneaux doit être un bouton, et celui-ci doit se trouver à l’intérieur du titre"; aussi ça assure des interactions au clavier qui sont totalement standard.

expliquer ce qu'on va faire différemment etc.

Pourquoi ?
Dans l'audit il est dit :

Motif de conception ARIA button : lorsqu’il n’est pas possible d’utiliser un élément
<button>, ARIA fournit un motif de conception button :
L’élément doit avoir un attribut role="button".
L’événement doit se déclencher sur le clic de souris, la touche Entrée et la touche
Espace.
Si l’élément ne peut pas recevoir le focus de tabulation, l’élément doit posséder une
propriété tabindex="0" pour le rendre opérable au clavier.

Et dans la page détaillant le motif de conception on peut y lire

The title of each accordion header is contained in an element with role button.

Conclusion, a accessibilité <button> et role="button" c'est kif kif d'un point de vue sémantique.

Donc on est bien libre de proposer ce qui est le plus pertinent avec notre existant.
Ma proposition est juste d'utiliser ce que tu as fait pour les cellules foldable.

Ce qui me dérange aussi dans ce lot de patch c'est le CSS pour générer un bouton "unset".
J'ai l'impression qu'utiliser un role="button" nous évite des contorsions CSS.

Après en effet, ce qui change par rapport au travail sur les cellules foldable et que je n'avais pas relevé c'est : "le bouton doit être au sein du h2".

Je comprends qu'utiliser un bouton assure des interactions au clavier qui sont totalement standard par défaut.
Mais là encore la function Js combo prepare_foldable fait déjà le job

Je suis donc partisan d'utiliser l'existant pour combo.

Enfin ce point ne semble pas être traité dans le patch :

The accordion header button element has aria-controls set to the ID of the element containing the accordion panel content.