Symbole erscheinen nicht in der Seitenleiste, dann ist die Farbe nicht definiert

Ich finde es toll, dass Icons jetzt im Sidebar angezeigt werden! :sunny: :sunglasses:

Was wie ein kleiner Glitch erscheint: Icons werden nicht angezeigt, wenn die Farbe nicht definiert ist. Z.B. auf diesem Screenshot hat die erste Kategorie ein definiertes Icon, aber keine Farbe. Dann wird das Icon nicht angezeigt:

Der Nachteil, wenn man allen Icons Farben zuweist, ist, dass die Icons dann in den Listenansichten nicht mit geerbten Farben angezeigt werden (Akzent, wenn das Thema ungelesen ist / gedimmt, wenn das Thema gelesen ist). Z.B. hier wird das Announcements-Icon mit geerbter Farbe angezeigt, während das Icon mit definierter Farbe immer gleich bleibt:

Ich denke, Icons sollten einfach im Sidebar mit der Standard-Icon-Farbe für den Sidebar angezeigt werden. Dann liegt es am Benutzer, ob er eine statische Farbe definieren möchte oder nicht.

4 „Gefällt mir“

Entschuldigung für die Verzögerung, @manuel, das ergibt Sinn. Ich habe dem Thema einen #pr-welcome-Tag hinzugefügt. Ich bin offen dafür, eine PR zu prüfen, die dieses Problem behebt. Danke!

3 „Gefällt mir“

Ich habe einen einfachen PR dafür erstellt: FIX: Render sidebar icons when no color is defined by nolosb · Pull Request #32 · discourse/discourse-category-icons · GitHub

7 „Gefällt mir“

Danke, die Korrektur wurde jetzt übernommen!

6 „Gefällt mir“

Leider rendert diese Änderung nun alle Icons mit definierten Farben in Grau.

Ich benutze

category,icon,categoryColour
2 „Gefällt mir“

Ja, jetzt wird es standardmäßig verwendet, wenn die Farbe leer gelassen wird. Mit dem Entfernen von categoryColor funktioniert es für mich.
wie hier: category,icon

2 „Gefällt mir“

Ja, aber es bricht die bestehenden Definitionen (ich habe einige).

@manuel Wären Sie so freundlich und würden den Test für categoryColo(u)r aufnehmen, damit wir nicht alle Definitionen in allen bestehenden Instanzen manuell korrigieren müssen? Vielen Dank im Voraus!

4 „Gefällt mir“

Ok, hier passiert einiges mit Farben :upside_down_face:

Die Deklaration ist tatsächlich sowohl bei den Standard-Badges als auch in der Seitenleiste veraltet. Nur bei den Standard-Badges schlägt sie still fehl, da sie auf einen Inline-Stil angewendet wird:

Habe einen PR für die Seitenleisten-Regression gemacht: FIX: Render legacy color declaration by nolosb · Pull Request #34 · discourse/discourse-category-icons · GitHub

4 „Gefällt mir“