الأيقونات لا تظهر في الشريط الجانبي إذا لم يتم تحديد اللون

أحب أن الأيقونات تظهر الآن في الشريط الجانبي! :sunny: :sunglasses:

ما يبدو أنه خلل بسيط: الأيقونات لا تظهر عندما لا يتم تحديد اللون. على سبيل المثال، في لقطة الشاشة هذه، تم تحديد أيقونة للفئة الأولى، ولكن بدون لون. ثم لا تظهر الأيقونة:

العيب في تحديد جميع الأيقونات بالألوان هو أن الأيقونات لن تظهر بعد ذلك بألوان موروثة في طرق عرض القائمة (لون مميز عندما يكون الموضوع غير مقروء / باهت عندما يكون الموضوع مقروءًا). على سبيل المثال، هنا تظهر أيقونة الإعلانات بلون موروث، بينما تظل الأيقونة ذات اللون المحدد دائمًا كما هي:

أعتقد أنه يجب أن تظهر الأيقونات ببساطة في الشريط الجانبي باللون الافتراضي للأيقونة للشريط الجانبي. ثم يعود الأمر للمستخدم إذا كان يريد تحديد لون ثابت أم لا.

4 إعجابات

عذرًا على التأخير هنا @manuel، هذا منطقي، لقد أضفت علامة pr-welcome إلى الموضوع، أنا منفتح على النظر في طلب سحب (PR) يصلح هذه المشكلة. شكرًا!

3 إعجابات

لقد قمت بطلب سحب بسيط لهذا: 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:

في الواقع، الإعلان قديم في كل من الشارات الافتراضية وفي الشريط الجانبي. فقط في الشارات الافتراضية، يفشل بصمت لأنه يتم تطبيقه على نمط مضمن:

لقد قمت بإنشاء طلب سحب للانحدار في الشريط الجانبي: FIX: Render legacy color declaration by nolosb · Pull Request #34 · discourse/discourse-category-icons · GitHub

4 إعجابات