Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(view): Primary color에 따라 File Summary 글자 색상 동적 변경 #558

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

lxxmnmn
Copy link
Contributor

@lxxmnmn lxxmnmn commented Jul 26, 2024

Related issue

#528

Result

File Summary 차트의 배경 색상이 되는 primary color가 변경될 때마다 글자 색상이 동적으로 변경됩니다.

githru#528

Work list

  • CSS의 filter 속성을 사용하여 배경 색상이 어두울 때는 white 컬러, 배경 색상이 밝을 때는 black 컬러로 보입니다.
  • 이슈에 첨부해주신 링크를 참고했습니다. 감사합니다. 🤗

Discussion

색상 값을 계산하거나 SVG 필터를 씌우는 것보다 간단한 방법이라고 생각해서 이렇게 구현했는데요,
혹시 더 좋은 방법이 있다면 편하게 제안해주세요! 🙌🙌🙌

Copy link
Contributor

@xxxjinn xxxjinn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

덕분에 새로운 속성 알아갑니다..!!! 개인적으로 글자 색이 white로 고정되어 있는 것 보다 좋은 것 같아요 👍

@@ -2,6 +2,7 @@

.file-icicle-summary {
text {
fill: $white;
fill: var(--primary-color);
filter: invert(100) grayscale(100) contrast(100);
Copy link
Contributor

@Sang-minKIM Sang-minKIM Jul 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

와.. 이런 방법이..!!
filter라는 속성이 있는 걸 처음 알았네요
진짜 좋은 방법인 것 같아요 많이 배워갑니다👍🏻

@lxxmnmn lxxmnmn merged commit c803e1b into githru:main Jul 26, 2024
2 checks passed
HIITMEMARIO pushed a commit to HIITMEMARIO/githru-vscode-ext that referenced this pull request Jul 28, 2024
@ytaek
Copy link
Contributor

ytaek commented Jul 28, 2024

와 이 방법 너무 멋진데요!!!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants