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

fix(view): 각 content title 수정 #585

Merged
merged 4 commits into from
Aug 2, 2024
Merged

fix(view): 각 content title 수정 #585

merged 4 commits into from
Aug 2, 2024

Conversation

xxxjinn
Copy link
Contributor

@xxxjinn xxxjinn commented Aug 1, 2024

Related issue

close #582

Result

image

Work list

  • 멘토님이 말씀하셨던 대로, file icicle summary에서 file summary로 text 변경했습니다.
  • 이전에는 App.tsx의 top-container에 FilteredAuthors가 속해있었는데 해당 컴포넌트는 middle-container의 VerticalClusterList와 관련이 있다고 생각되어 VerticalClusterList 내부에 두는 것으로 코드를 수정했습니다. 혹시 다른 의견이 있으시다면 남겨주세요..!!
  • 첨부한 사진에서 표시한 부분인 ' Authors: ' text도 추가했습니다.
  • middle container의 padding 값을 각 내부의 content에서 주는 것이 아닌, middle container 전체에 주도록 스타일 수정했습니다.

Discussion

@xxxjinn xxxjinn self-assigned this Aug 1, 2024
@xxxjinn xxxjinn requested review from a team as code owners August 1, 2024 15:01
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.

깔끔해졌네요!

충돌은 너무 걱정하지 않으셔도 됩니다. 왠만하면 auto merge 될 거고...
그리고, 먼저 머지하는 사람이 임자입니다 😈😈😈😈😈😈

@@ -35,5 +35,6 @@ body {
display: grid;
grid-template-columns: 4fr 1fr;
height: calc(100vh - 200px);
margin-top: 20px;

Copy link
Contributor

Choose a reason for hiding this comment

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

안녕하세요! 해당 빈칸은 margin 삭제하다가 생긴것같은데 필요없다면 삭제하는게 좋을것같아요 :)

Copy link
Contributor

@Sang-minKIM Sang-minKIM left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~ 👍🏻👍🏻

@pcwadarong pcwadarong merged commit c2832ec into githru:main Aug 2, 2024
2 checks passed
@pcwadarong
Copy link
Member

엇 머지는 보통 본인이 할까요..? 제가 merge 허용된 버튼이 있길래 해드리면 좋을 것 같아 눌러버렸는데 본인이 하셔야 하는데 제가 괜시리 참여한 게 아닌가 싶네요 ㅠㅠ 죄송합니닷,, 수고하셨습니다!!!! 👍🏻👍🏻 👍🏻👍🏻 👍🏻👍🏻

@xxxjinn
Copy link
Contributor Author

xxxjinn commented Aug 2, 2024

엇 머지는 보통 본인이 할까요..? 제가 merge 허용된 버튼이 있길래 해드리면 좋을 것 같아 눌러버렸는데 본인이 하셔야 하는데 제가 괜시리 참여한 게 아닌가 싶네요 ㅠㅠ 죄송합니닷,, 수고하셨습니다!!!! 👍🏻👍🏻 👍🏻👍🏻 👍🏻👍🏻

괜찮습니다ㅎㅎ 감사합니다!!!

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

Successfully merging this pull request may close these issues.

[view] 각 content title 설정 text 수정
5 participants