Mise à niveau de la version de Discourse causant une perte de données (aria-label) sur les propriétés des éléments DOM

Soutien
1

Salut,

Nous avons personnalisé le site discourse par défaut dans les versions de Discourse comprises entre 2.3 et 2.6. Dans les versions mentionnées, nous avons appliqué du CSS personnalisé pour masquer le bouton de réponse dans les boutons du pied de page de la page de sujet en fonction de la valeur de la propriété d’élément aria-label.

Après la mise à niveau de la version de Discourse vers 2.8.0.beta6, la propriété aria-label était manquante pour l’élément DOM du bouton de réponse. Cependant, d’autres boutons tels que partager, marquer, signaler, etc., possèdent toujours la propriété aria-label dans les boutons du pied de page de la page de sujet. En raison de ce comportement, nous rencontrons un problème.

Avant la mise à niveau vers la version 2.8.0 :

With_aria-label
With_aria-label1492×132 15.8 KB

Après la mise à niveau vers la version 2.8.0 :

Without_aria-label
Without_aria-label1501×124 15.4 KB

La mise à niveau de la version 2.8.0.beta6 de Discourse a-t-elle modifié ou ignoré des propriétés dans les éléments DOM de la page ?

1 « J'aime »
3

Vous avez raison, il semble qu’il manque.\n\nIl semble que la valeur de aria-label soit calculée ici : discourse/register-topic-footer-button.js at 362c47ce6a87783808cb4417912a75250f003bf8 · discourse/discourse (github.com)\n\nIl semble qu’il y ait eu du travail sur cela il y a seulement 6 jours ? :\n\nDEV: ajoute une nouvelle API de menu déroulant de pied de page de sujet par jjaffeux · Pull Request #14747 · discourse/discourse (github.com)

1 « J'aime »
4

Bien que le bouton de réponse ne provienne pas de là, il s’agit d’un {{d-button}} autonome

https://github.com/discourse/discourse/blob/main/app/assets/javascripts/discourse/app/templates/components/topic-footer-buttons.hbs#L59

Le changement provient de ce commit :

Je suis désolé, mais nous n’avons aucune raison de réintégrer cet aria-label ici, le style basé sur l’aria-label n’est pas une bonne pratique.

2 « J'aime »
5

Est-ce parce que la valeur dépend de la locale ?

6

Oui exactement, c’est aussi très sujet à changement :+1:

1 « J'aime »
7

Ce sujet a été automatiquement fermé 30 jours après la dernière réponse. Les nouvelles réponses ne sont plus autorisées.