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): commit-item 요소를 한줄로 출력 #371

Merged
merged 1 commit into from
Aug 6, 2023

Conversation

KKanghh
Copy link
Contributor

@KKanghh KKanghh commented Aug 2, 2023

Related issue

resolve #291

Result

스크린샷 2023-08-02 오후 7 32 52

Work list

.commit-item의 display를 flex로, justify-content를 space-between으로 설정하여 .author-date가 우측에 정렬되도록 설정.
.author-date가 한줄로 출력되도록 설정
.commit-id의 폭을 고정된 값으로 설정하여 .author-date가 보기 좋게 정렬 되도록 설정

Discussion

@KKanghh KKanghh self-assigned this Aug 2, 2023
@KKanghh KKanghh requested review from a team as code owners August 2, 2023 10:44
Copy link
Contributor

@newgardener newgardener left a comment

Choose a reason for hiding this comment

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

🍯🍯👍🏻
작은 개선인데 보기가 훨씬 좋아졌네요!!

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.

빨리 해결되었네요!!

추가로 아래 건에 대해서 view팀이 같이 논의 해보면 좋을 것 같습니다.

  • 테스트할 대표 환경 정의
    : 기본으로 할 width의 디폴트 값 (vscode에서 띄울 때랑, npm run server로 할 때랑 width가 보통 다르게 테스트 되네요. 지금 pr에 올려주신 이미지도 width가 좀 긴 편인듯?)
    : 혹은 기본으로 할 width의 범위: 반응형으로 구현한다면, 범위를 어떻게까지 해야 할지?
    : 그 외 필요한 것들(이 있지 않을까요/ ^^:)
  • message가 길 때 ...으로 줄어드는데, 이걸 더 잘 보여줄 수 있는 or hover시에 보여주는 방법 추가.

@KKanghh 님, 괜찮으시면 위 두 안건에 대해 각각 이슈로 만들어주시면 감사하겠습니다!

@KKanghh
Copy link
Contributor Author

KKanghh commented Aug 3, 2023

@ytaek 감사합니다! 지금 가족휴가와서 주말중으로 해도 될까요??

@KKanghh KKanghh merged commit b747aca into githru:main Aug 6, 2023
2 checks passed
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.

[view] 디테일 컴포넌트에서 commit-item 요소를 한 줄로 표현하기
3 participants