サイドバーにアイコンが表示されない、色が定義されていない

サイドバーにアイコンが表示されるようになったのは素晴らしいですね! :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