색상이 정의되지 않으면 사이드바에 아이콘이 표시되지 않음

사이드바에 아이콘이 표시되니 좋네요! :sunny: :sunglasses:

작은 버그처럼 보이는 부분: 색상이 정의되지 않은 경우 아이콘이 표시되지 않습니다. 예를 들어, 아래 스크린샷에서 첫 번째 카테고리에는 아이콘이 정의되어 있지만 색상은 없습니다. 이 경우 아이콘이 표시되지 않습니다:

모든 아이콘에 색상을 정의하는 단점은, 목록 뷰에서 상속된 색상(주제가 읽지 않은 상태일 때 강조 색상, 읽은 상태일 때 흐려진 색상)으로 아이콘이 표시되지 않는다는 점입니다. 예를 들어, 여기서는 공지사항 아이콘이 상속된 색상으로 표시되는 반면, 색상이 정의된 아이콘은 항상 동일한 색상으로 유지됩니다:

아이콘은 기본적으로 사이드바의 기본 아이콘 색상으로 표시되어야 한다고 생각합니다. 그런 다음 사용자가 정적 색상을 정의할지 여부는 사용자의 선택에 맡기면 됩니다.

4개의 좋아요

Sorry for the delay here @manuel, this makes sense, I added a pr-welcome tag to the topic, I’m open to looking at a PR that fixes this issue. Thanks!

3개의 좋아요

이 건에 대해 간단한 PR을 올렸습니다: FIX: Render sidebar icons when no color is defined - 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:

사실 이 선언은 기본 배지와 사이드바 양쪽 모두에서 더 이상 필요하지 않습니다. 다만 기본 배지에서는 인라인 스타일에 적용되므로 오류가 조용히 무시됩니다:

사이드바 회귀 문제(regression)에 대한 PR을 만들었습니다: FIX: Render legacy color declaration - Pull Request #34 - discourse/discourse-category-icons - GitHub

4개의 좋아요