manuel
(Manuel Kostka)
6월 15, 2023, 3:22오후
1
사이드바에 아이콘이 표시되니 좋네요!
작은 버그처럼 보이는 부분: 색상이 정의되지 않은 경우 아이콘이 표시되지 않습니다. 예를 들어, 아래 스크린샷에서 첫 번째 카테고리에는 아이콘이 정의되어 있지만 색상은 없습니다. 이 경우 아이콘이 표시되지 않습니다:
모든 아이콘에 색상을 정의하는 단점은, 목록 뷰에서 상속된 색상(주제가 읽지 않은 상태일 때 강조 색상, 읽은 상태일 때 흐려진 색상)으로 아이콘이 표시되지 않는다는 점입니다. 예를 들어, 여기서는 공지사항 아이콘이 상속된 색상으로 표시되는 반면, 색상이 정의된 아이콘은 항상 동일한 색상으로 유지됩니다:
아이콘은 기본적으로 사이드바의 기본 아이콘 색상으로 표시되어야 한다고 생각합니다. 그런 다음 사용자가 정적 색상을 정의할지 여부는 사용자의 선택에 맡기면 됩니다.
4개의 좋아요
pmusaraj
(Penar Musaraj)
9월 5, 2023, 3:19오후
3
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개의 좋아요
manuel
(Manuel Kostka)
6월 14, 2024, 10:49오전
4
7개의 좋아요
Manuel Kostka:
이 부분에 대해 간단한 PR을 올렸습니다
불행히도 이 변경 사항으로 인해 색상이 정의된 모든 아이콘이 회색으로 표시됩니다.
저는 다음을 사용하고 있습니다.
category,icon,categoryColour
2개의 좋아요
Don
6월 15, 2024, 6:45오후
7
네, 이제 색상을 비워두면 기본적으로 이걸 사용해요. categoryColor를 제거하면 제 환경에서는 잘 동작합니다.
이렇게요: category,icon
2개의 좋아요
네, 하지만 기존 정의들(제가 꽤 많이 가지고 있습니다)이 깨져 버립니다.
@manuel categoryColo(u)r에 대한 테스트를 포함해 주실 수 있을까요? 그래야 기존 인스턴스의 모든 정의를 수동으로 수정할 필요가 없게 됩니다. 미리 감사드립니다!
4개의 좋아요
manuel
(Manuel Kostka)
6월 15, 2024, 9:16오후
9
음, 여기 색상 관련해서 꽤 많은 일이 일어나고 있네요
사실 이 선언은 기본 배지와 사이드바 양쪽 모두에서 더 이상 필요하지 않습니다. 다만 기본 배지에서는 인라인 스타일에 적용되므로 오류가 조용히 무시됩니다:
사이드바 회귀 문제(regression)에 대한 PR을 만들었습니다: FIX: Render legacy color declaration - Pull Request #34 - discourse/discourse-category-icons - GitHub
4개의 좋아요