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

[view] Summary author background-color 지정 #102

Closed
1 task done
jejecrunch opened this issue Sep 7, 2022 · 7 comments
Closed
1 task done

[view] Summary author background-color 지정 #102

jejecrunch opened this issue Sep 7, 2022 · 7 comments
Assignees
Labels

Comments

@jejecrunch
Copy link
Contributor

jejecrunch commented Sep 7, 2022

관련 PR

#97

현 상황

전체적으로 파란계열이라고 임시적으로 컬러 계열을 정한 상황
author가 다른 클러스터에서도 같은 이름인데 다른 색이 나오는 경우가 있음

해야할 일

  • 컬러 계열에 맞춰 나오게 변경 -> 현 상황의 2번 문제도 해결 될 것 같음
@jejecrunch jejecrunch self-assigned this Sep 7, 2022
@jejecrunch
Copy link
Contributor Author

jejecrunch commented Sep 10, 2022

현 상황

컬러 계열에 맞춰 나오게 다음과 같은 배열을 사용했습니다.

[
  "00ADF7",
  "0077AA",
  "4AC3F7",
  "0BD9E0",
  "33C2FF",
  "4BC4F9",
  "0089C4",
];

색은 다음과 같습니다.

image

image

또 다른 문제

@ytaek 님께서 웹뷰로 보이는 화면을 올려주셨는데 색이 좀 많이 튀는 것 같아서 변경할 필요가 있을 것 같습니다.

image

해야할 일

  • 웹뷰로 실행해보면서 비슷한 계열의 색 조합 찾아서 적용

@ytaek
Copy link
Contributor

ytaek commented Sep 10, 2022

오 그렇네요. 색깔 부분은 배포 전에 빠르게 수정해보면 좋을 것 같습니다!!
밝은색은 까만 계열 바탕에서 더 대비가 되니까, 적당히 톤 다운 된 놈으로 고르는게 좋겠습니다.

@jejecrunch
Copy link
Contributor Author

하 요 부분 어떻게 수정해야할지 애매하네요 ...
제너레이터로 shade 헥스 값 가져와서 뿌렸는데 색이 한 색이다보니 다른 사람이라는 게 거의 티가 안나서 ㅜㅜ

우선 아래 이미지에서 빨간색으로 동그라미친 색으로 PR 올려두겠습니다 !

jejecrunch added a commit to jejecrunch/githru-vscode-ext that referenced this issue Sep 12, 2022
@ytaek
Copy link
Contributor

ytaek commented Sep 12, 2022

하 요 부분 어떻게 수정해야할지 애매하네요 ... 제너레이터로 shade 헥스 값 가져와서 뿌렸는데 색이 한 색이다보니 다른 사람이라는 게 거의 티가 안나서 ㅜㅜ

아, #151 에도 리뷰를 달긴했는데 조금 색이 밝은 계열이라 다른 set으로 하는 것도 좋을 것 같구요.
그리고, author라는 categorical value를 구분하는 용도로 지금

  • avatar원의 배경색깔
  • avatar원 안의 영어 1글자.

2가지로 인코딩을 하고 있는데요.

  • 색깔(Hue)로 명확하게 author를 구분하는건 author가 10명 이하 정도 되어야할 때인데, author가 많을 때에도 color encoding을 쓰는게 맞는지
  • category를 구분하는 용도로 색깔이 중요한 channel인데, 이걸 author에 몰빵하는게 맞는지
    도 좀 고민해보면 좋을 것 같긴 하구요.
    (git kraken같은 경우는 색을 branch에 할당하고 있어요)

image

여러가지 고민을 해보면 좋을 것 같습니다 :- ) 천천히 해보시죠 ㅎㅎ

당장은 위에서 말했던대로 tone down 된 색으로 하시면 괜찮을 것 같습니다.

@jejecrunch
Copy link
Contributor Author

초반에 원 안에 들어갈 이미지를 줄 수 없다고 하셔가지고 컬러라는 임시적인 방편을 사용한 거라서 프로필의 이미지를 가져와주실 수 있는지에 대해서 엔진팀과도 더 얘기해봐야할 것 같습니다. 그리고 브랜치 별로 색을 할당한다면 또 어떻게 해야할지? githru의 경우에는 브랜치 별이 아니라 클러스터별로 나누어져있는데 클러스터별로 나누어야 하는지?에 대해서도 고민이 되네요. 저도 author가 너무 많아질 때가 걱정이긴 했습니다만 우선 0.1 버전에는 성과물 위주라고 생각해서 ㅎㅎ 좀 더 고민해보고 변경하면 좋을 것 같습니다 !! 피드백 감사합니다 !!

전체 색은 다음과 같고
image

적용된 이미지는 다음과 같습니다.
image

jejecrunch added a commit that referenced this issue Sep 12, 2022
* chore(view): Run `npm install`

* chore(view): Run `npm install`

* fix(view): fix summary issues #146

* fix(view): modify each node height

* fix(view): modify author background color #102

* fix(view): modify long commit message
@jejecrunch jejecrunch added this to the v0.2.0 milestone Sep 15, 2022
@ytaek
Copy link
Contributor

ytaek commented Sep 21, 2022

이 문제는 engine에서 avatar를 가져오게 되면
적어도 author color는 해결되지 않을까요? 😸

avatar로 해결된다면 close 해주세요~ 일단 v0.2.0 tag는 제외하겠습니다.

@ytaek ytaek removed this from the v0.2.0 milestone Sep 21, 2022
@hanseul-lee
Copy link
Contributor

#256 에서 avatar 적용하여 이 이슈 close 하겠습니다🙇‍♀️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Done
Development

No branches or pull requests

3 participants