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] 클러스터 리스트 y축 스크롤바 얇게 보이는 문제 해결 #729

Conversation

jkea1
Copy link
Contributor

@jkea1 jkea1 commented Sep 19, 2024

Related issue

issue 728

Result

Screenshot 2024-09-19 at 9 06 29 PM

Work list

클러스터 리스트의 스크롤바가 거의 보이지 않을 정도로 얇게 보이는 문제를 발견했습니다.
스크롤바의 thumb 부분의 width는 _reset.scss 파일에서 전역으로 관리 되고 있습니다.

*::-webkit-scrollbar {
    width: 8px;
  }

그렇다면 width = 8px; 이 일괄 적용되어야 하는데 스크롤바의 굵기가 너무 얇아 보였습니다.

클러스터 리스트의 body 부분의 오른쪽의 padding이 부족하여 스크롤바의 thumb 부분이 가려져 있어 생긴 문제임을 파악했습니다.
그래서 여분의 padding을 주어 스크롤바가 충분히 보이도록 스타일을 수정해 주었습니다.

Discussion

@jkea1 jkea1 added this to the v0.7.2 milestone Sep 19, 2024
@jkea1 jkea1 self-assigned this Sep 19, 2024
@jkea1 jkea1 requested review from a team as code owners September 19, 2024 12:22
@xxxjinn
Copy link
Contributor

xxxjinn commented Sep 21, 2024

image 브랜치 명에 한글이 들어가서 그런가 사진과 같은 창이 뜨고 있는 것을 발견했습니다 !.! 찾아보니 첨부드린 링크를 보면 머지하는 데에 문제는 없는 것 같은데 평소에도 브랜치 명에 한글이 들어가도록 이름을 지으시는 경우가 있나용?? 저는 경험이 없어서 궁금해서 여쭤봅니다!!

https://github.com/orgs/community/discussions/51906#discussioncomment-5870895

@ytaek
Copy link
Contributor

ytaek commented Sep 27, 2024

image 브랜치 명에 한글이 들어가서 그런가 사진과 같은 창이 뜨고 있는 것을 발견했습니다 !.! 찾아보니 첨부드린 링크를 보면 머지하는 데에 문제는 없는 것 같은데 평소에도 브랜치 명에 한글이 들어가도록 이름을 지으시는 경우가 있나용?? 저는 경험이 없어서 궁금해서 여쭤봅니다!!
https://github.com/orgs/community/discussions/51906#discussioncomment-5870895

저도 잘 모르지만, 궁금하네요 ㅎㅎ
어차피 pr merge되면 사라질 값들이라 상관은 없을 것 같습니다 : )

ytaek
ytaek previously approved these changes Sep 27, 2024
Copy link
Contributor

@ytaek ytaek left a comment

Choose a reason for hiding this comment

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

세심한 수정 감사합니다! LGGGTM! 👍👍👍

HIITMEMARIO
HIITMEMARIO previously approved these changes Sep 27, 2024
Copy link
Contributor

@HIITMEMARIO HIITMEMARIO left a comment

Choose a reason for hiding this comment

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

수정 감사합니다!!! LGTM!

Sang-minKIM
Sang-minKIM previously approved these changes Sep 27, 2024
@JungJinkyung
Copy link

branch명을 이슈명으로 하면 좀 더 파악하기 편해서 그렇게하는 습관이 있습니다! 그런 문구가 뜨는 군요!? 머지하는데 문제가 없어서 다행입니다!

@jkea1 jkea1 dismissed stale reviews from Sang-minKIM, HIITMEMARIO, and ytaek via 6e516d4 October 7, 2024 12:49
Copy link
Contributor

@ytaek ytaek left a comment

Choose a reason for hiding this comment

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

LGTM!

@ytaek ytaek modified the milestones: v0.7.2, v0.7.3 Oct 16, 2024
@ytaek
Copy link
Contributor

ytaek commented Oct 16, 2024

@jkea1 님, 반영 완료면 merge 부탁드립니다!!

@ytaek
Copy link
Contributor

ytaek commented Nov 9, 2024

고생하셨습니다. 머지 부탁드릴께요~

@ytaek
Copy link
Contributor

ytaek commented Nov 20, 2024

머지 하겠습니다.

@ytaek ytaek merged commit dc12d06 into main Nov 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[view] vertical cluster list의 y축 스크롤 바 굵기가 얇게 보이고 있는 문제
7 participants