Los iconos no aparecen en la barra lateral, entonces el color no está definido

¡Me encanta que los iconos ahora se muestren en la barra lateral! :sunny: :sunglasses:

Lo que parece un pequeño error: Los iconos no se muestran cuando no se define el color. Por ejemplo, en esta captura de pantalla, la primera categoría tiene un icono definido, pero sin color. Entonces, el icono no se muestra:

La desventaja de definir todos los iconos con colores es que los iconos no se mostrarán con colores heredados en las vistas de lista (acentuado cuando el tema no ha sido leído / atenuado cuando el tema ha sido leído). Por ejemplo, aquí el icono de Anuncios se muestra con color heredado, mientras que el icono con color definido siempre permanece igual:

Creo que los iconos simplemente deberían mostrarse en la barra lateral con el color de icono predeterminado para la barra lateral. Luego, depende del usuario si desea definir un color estático o no.

4 Me gusta

Disculpa la demora, @manuel, tiene sentido. Añadí una etiqueta pr-welcome al tema, estoy abierto a revisar una PR que solucione este problema. ¡Gracias!

3 Me gusta

Hice una PR sencilla para esto: FIX: Render sidebar icons when no color is defined by nolosb · Pull Request #32 · discourse/discourse-category-icons · GitHub

7 Me gusta

¡Gracias, la corrección ya está fusionada!

6 Me gusta

Desafortunadamente, este cambio ahora renderiza todos los iconos con colores definidos en gris.

Estoy usando

category,icon,categoryColour
2 Me gusta

Sí, ahora lo usa por defecto si dejas vacío el color. Con quitar categoryColor funciona para mí.
como esto: category,icon

2 Me gusta

Sí, pero rompe las definiciones existentes (tengo bastantes).

@manuel ¿Serías tan amable de incluir la prueba para categoryColo(u)r para que no necesitemos arreglar manualmente todas las definiciones en todas las instancias existentes? ¡Gracias de antemano!

4 Me gusta

Ok, hay bastantes cosas sucediendo aquí con los colores :upside_down_face:

En realidad, la declaración está obsoleta tanto en las insignias predeterminadas como en la barra lateral. Solo en las insignias predeterminadas falla silenciosamente ya que se aplica a un estilo en línea:

Hice una PR para la regresión de la barra lateral: FIX: Render legacy color declaration by nolosb · Pull Request #34 · discourse/discourse-category-icons · GitHub

4 Me gusta