Actualización de versión de Discourse causa pérdida de datos (aria-label) en propiedades de elementos DOM

Hola,

Hemos personalizado el sitio predeterminado de Discourse en la versión de Discourse entre (2.3 - 2.6). En las versiones mencionadas, hemos aplicado CSS personalizado para ocultar el botón de respuesta en los botones del pie de página de la página del tema basándonos en el valor de la propiedad del elemento aria-label.

Después de actualizar la versión de Discourse a 2.8.0.beta6, faltaba la propiedad aria-label para el elemento DOM del botón de respuesta. Pero otros botones como compartir, marcar, marcar como spam, etc., todavía tienen la propiedad aria-label en los botones del pie de página del tema. Debido a este comportamiento, estamos experimentando un problema.

Antes de actualizar a la versión 2.8.0:

Después de actualizar a la versión 2.8.0

¿La actualización de la versión 2.8.0.beta6 de Discourse modificó o ignoró alguna propiedad en los elementos DOM de la página?

1 me gusta

Tienes razón, parece que falta.

Parece que el valor de aria-label se calcula aquí: discourse/register-topic-footer-button.js at 362c47ce6a87783808cb4417912a75250f003bf8 · discourse/discourse (github.com)

¿Parece que hubo trabajo en esto hace solo 6 días?:

DEV: añade una nueva API de menú desplegable para el pie de página de temas por jjaffeux · Pull Request #14747 · discourse/discourse (github.com)

1 me gusta

Aunque el botón de respuesta no proviene de aquí, es un {{d-button}} independiente

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

El cambio proviene de este commit:

Lo siento, pero no tenemos ninguna razón para volver a añadir esta etiqueta aria aquí, el estilo basado en la etiqueta aria no es una buena práctica.

2 Me gusta

¿Es porque el valor depende de la configuración regional?

Sí, exactamente, eso también es muy propenso a cambiar :+1:

1 me gusta

Este tema se cerró automáticamente 30 días después de la última respuesta. Ya no se permiten nuevas respuestas.