[view] 툴팁 교체 및 폴더 구조 그래프 명시성 확보 #720
Conversation
There was a problem hiding this comment.
mui로 교체하였기에 삭제한 부분입니다.
| @@ -80,7 +81,7 @@ const Detail = ({ selectedData, clusterId, authSrcMap }: DetailProps) => { | |||
| <span className="message">{message}</span> | |||
| </div> | |||
| </div> | |||
| <span className="author-date"> | |||
| <span className="commit-date"> | |||
There was a problem hiding this comment.
해당 container가 commit을 다루고 있고, 실제 날짜도 author에 관련되어 있다기 보다 커밋한 일시 자체를 다루고 있기 때문에 더 알맞다고 생각하여 수정하였습니다.
| <Tooltip | ||
| placement="right" | ||
| title={id} | ||
| > | ||
| <p>{`${id.slice(0, 6)}...`}</p> | ||
| </Tooltip> |
There was a problem hiding this comment.
기존처럼 6개만 잘라 내용을 보여주고 툴팁에 전체를 담는 형식을 유지하되, 말 줄임 표시로 ... 을 넣었습니다.
There was a problem hiding this comment.
git 사용 시 보통 6a58676 요렇게 커밋 id를 6자리로 끊기도 해서 저는 원래대로 ... 없이 가는게 조금 더 깔끔한 것 같기도 한데 다른 분들 생각은 어떠신지 궁금하네요..!!!
There was a problem hiding this comment.
앗 이제 봤네요!! 다음 pr 때 수정하겠습니다!
| p { | ||
| font-size: $font-size-title; | ||
| font-weight: $font-weight-semibold; | ||
| } | ||
| .author-bar-chart__title { | ||
| font-size: $font-size-title; | ||
| font-weight: $font-weight-semibold; | ||
| } |
There was a problem hiding this comment.
기존에 컨테이너 박스 안의 모든 p의 폰트 사이즈와 두께를 결정하고 있는 관계로, 아래에서 다시 .name 의 두께를 정해주어도 변경되지 않았습니다. 이전에 작성한 의도에 맞게 name만 진한 굵기로, 나머지는 보통 굵기로 나오도록 수정하기 위해 따로 class 명을 주었습니다.
| @@ -40,7 +40,6 @@ | |||
|
|
|||
| .axis { | |||
| color: $color-white; | |||
| font-weight: $font-weight-semibold; | |||
There was a problem hiding this comment.
그래프의 label에 해당하는 부분입니다. 상단 기간 그래프의 날짜 부분이 보통 굵기로 설정되어 있었기에 통일하였습니다.
There was a problem hiding this comment.
깔끔하게 정리되었네요!! LGGGGGGGGGTM!
참고로, PR의 title 및 commit message 의 경우에는 conventional commit 형태를 이용해주시면 감사하겠습니다!
https://github.com/githru/githru-vscode-ext/blob/main/CONTRIBUTING.md#commit-message
| line-height: 1.5; | ||
| border-radius: 5px; | ||
| color: $color-dark-gray; | ||
| background: $color-medium-gray; |
There was a problem hiding this comment.
혹싱 요 툴팁이랑 commit-id의 툴팁이랑 색이 같은가요 ?.? 툴팁의 배경 색도 통일되면 좋을 것 같은데 사진 첨부해주신 건 뭔가 다른 것 싶어가지구 궁금합니다!!
| :root { | ||
| --primary-color: #ff8272; | ||
| } | ||
|
|
| title={id} | ||
| PopperProps={{ sx: { ".MuiTooltip-tooltip": { bgcolor: "#3c4048" } } }} |
| filter: invert(100) grayscale(100) contrast(100); | ||
| font-weight: $font-weight-semibold; |
There was a problem hiding this comment.
폴더 이름이 옅어서 안 보이는 느낌이라 컬러를 삭제하고 굵기를 조정하였습니다
pcwadarong
changed the title
Related issue
close #638
Result
Work list