Значки не отображаются в боковой панели, и цвет не задан

Отлично, что иконки теперь отображаются на боковой панели! :sunny: :sunglasses:

Кажется, есть небольшая неполадка: иконки не показываются, если цвет не задан. Например, на этом скриншоте у первой категории задана иконка, но нет цвета. В результате иконка не отображается:

Минус в том, что если задать цвет для всех иконок, они перестанут наследовать цвет в списках тем (акцентный цвет для непрочитанных тем и приглушенный для прочитанных). Например, здесь иконка «Объявления» отображается с наследуемым цветом, тогда как иконка с заданным цветом всегда остаётся неизменной:

По-моему, иконки на боковой панели должны просто отображаться с цветом иконки по умолчанию для панели. А дальше уже пользователь решает, хочет ли он задать статический цвет или нет.

4 лайка

Извините за задержку, @manuel. Всё логично, я добавил тег pr-welcome к теме. Я готов рассмотреть PR, который исправит эту проблему. Спасибо!

3 лайка

Сделал простой PR по этому вопросу: FIX: Render sidebar icons when no color is defined by nolosb · Pull Request #32 · discourse/discourse-category-icons · GitHub

7 лайков

Спасибо, исправление теперь влитано!

6 лайков

К сожалению, это изменение теперь отображает все иконки с заданными цветами в сером цвете.

Я использую

category,icon,categoryColour
2 лайка

Да, теперь по умолчанию используется цвет, если вы оставите поле цвета пустым. Удаление categoryColor работает у меня.
Так: category,icon

2 лайка

Да, но это ломает существующие определения (у меня их довольно много).

@manuel, не могли бы вы добавить проверку для categoryColo(u)r, чтобы нам не пришлось вручную исправлять все определения во всех существующих экземплярах? Заранее спасибо!

4 лайка

Хорошо, здесь происходит довольно много всего с цветами :upside_down_face:

На самом деле объявление устарело как для стандартных бейджей, так и для боковой панели. Только на стандартных бейджах оно терпит неудачу незаметно, так как применяется к встроенному стилю:

Сделал PR для регрессии в боковой панели: FIX: Render legacy color declaration by nolosb · Pull Request #34 · discourse/discourse-category-icons · GitHub

4 лайка