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

[Wip] 공통 컴포넌트 제작(6) PostListItem, TextIconButton(수정) #26

Merged
merged 10 commits into from
Jan 3, 2024

Conversation

loevray
Copy link
Collaborator

@loevray loevray commented Jan 2, 2024

작업 사항

  • 글 목록에서 사용할 PostListItem 컴포넌트를 제작하였습니다(스토리북 포함)
  • TextIconButton의 text방향 지정 및 텍스트 스타일 지정 등 유연성을 더 늘렸습니다.

관련 이슈

#7 6번째 태스크입니다

PR Point

  • props네이밍이 적절한가요?
  • 말줄임표를 위해 일단 제목과 내용너비를 200px로 설정해두었습니다. calc를 사용하려했는데 잘 안먹히네요..ㅠㅠ
  • 여러 chakra-ui로 이루어진 컴포넌트의 props를 전달할때 혹시 이런 방법은 어떨까요?
const Component = ({flexStyle, textStyle, iconStyle, etcProps}) => {
  <Flex {...flexStyle}>
   <Text {...textStyle}>
   <Icon {...IconStyle}>
   ...
}

컴포넌트 props의 덩치가 너무 커지는건 아닌가 고민됩니다ㅎㅎ

  • 이번에는 PostListItem이라고 명명해보았습니다. 제가 PostList컴포넌트까지 아예 react-query를 이용해서 만드는것도 하나의 방법일 것 같은데요, 어떻게 생각하시는지 궁금합니다. 또한 그렇게된다면 이전에 작성했던 UserList컴포넌트를 UserListItem 컴포넌트로 바꾼 뒤, UserList컴포넌트를 제작해 react-query를 넣어보겠습니다

참고사항

재사용 가능하게 만든다는 것이 참으로 어려운 것 같습니다! 이번에 TextIconButton도 꽤 수정사항이 생겨서 같이 봐주시면 감사하겠습니다.

image

@loevray loevray added 💄style UI/스타일 수정 👀 리뷰 필요 labels Jan 2, 2024
@loevray loevray self-assigned this Jan 2, 2024
@loevray loevray changed the title [Wip] 공통 컴포넌트 제작(6) PostListItem [Wip] 공통 컴포넌트 제작(6) PostListItem, TextIconButton(수정) Jan 2, 2024
Copy link
Collaborator

@whdgur5717 whdgur5717 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

  • props 네이밍은 딱히 이해 안가는 부분이 없었던 것 같습니다!
  • 제안해주신 전달방법은 굉장히 좋은 것 같습니다. 다같이 도입해보면 코드가 한층 깔끔해질 것 같습니다

src/components/common/PostListItem/index.tsx Show resolved Hide resolved
Copy link
Collaborator

@JaeHyunGround JaeHyunGround left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 !!

props 전달 방법도 좋은 것 같아요 !!

@loevray loevray merged commit 4368d53 into dev Jan 3, 2024
@loevray loevray deleted the feature/commonComponentPostListItem branch January 3, 2024 02:43
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.

3 participants