Os ícones não aparecem na barra lateral e, então, a cor não está definida

Adoro que os ícones agora apareçam na barra lateral! :sunny: :sunglasses:

O que parece um pequeno glitch: Ícones não aparecem quando a cor não é definida. Por exemplo, nesta captura de tela, a primeira categoria tem um ícone definido, mas sem cor. Então o ícone não é mostrado:

A desvantagem de definir todos os ícones com cores é que os ícones não aparecerão com cores herdadas nas visualizações de lista (destaque quando o tópico está não lido/diminuído quando o tópico está lido). Por exemplo, aqui o ícone de Anúncios aparece com cor herdada, enquanto o ícone com cor definida permanece sempre o mesmo:

Acho que os ícones deveriam apenas aparecer na barra lateral com a cor padrão do ícone para a barra lateral. Então, cabe ao usuário decidir se quer definir uma cor estática ou não.

4 curtidas

Desculpe pelo atraso, @manuel, isso faz sentido. Adicionei uma tag pr-welcome ao tópico, estou aberto a analisar um PR que corrija este problema. Obrigado!

3 curtidas

Fiz um PR simples para isto: FIX: Render sidebar icons when no color is defined by nolosb · Pull Request #32 · discourse/discourse-category-icons · GitHub

7 curtidas

Obrigado, a correção foi mesclada!

6 curtidas

Infelizmente, essa alteração agora renderiza todos os ícones com cores definidas em cinza.

Estou usando

category,icon,categoryColour
2 curtidas

Sim, agora ele usa por padrão se você deixar a cor vazia. Com a remoção de categoryColor funciona para mim.
como isto: category,icon

2 curtidas

Sim, mas isso quebra as definições existentes (tenho várias).

@manuel Você seria tão gentil e incluiria o teste para categoryColo(u)r para que não precisemos corrigir manualmente todas as definições em todas as instâncias existentes? Agradeço antecipadamente!

4 curtidas

Ok, há bastante coisa acontecendo aqui com cores :upside_down_face:

Na verdade, a declaração está obsoleta tanto nos emblemas padrão quanto na barra lateral. Apenas nos emblemas padrão, ela falha silenciosamente, pois é aplicada a um estilo inline:

Fiz um PR para a regressão da barra lateral: FIX: Render legacy color declaration by nolosb · Pull Request #34 · discourse/discourse-category-icons · GitHub

4 curtidas