Skip to content

Conversation

@gustn99
Copy link
Collaborator

@gustn99 gustn99 commented Dec 26, 2024

주요 작업 내용

  • BottomButton, PostBase 제외 모든 공통 컴포넌트 리팩토링 마쳤습니다.
  • 모든 파일에 대해 import문, 코드 흐름, 변수 명명 규칙, svg 컴포넌트 관련 작업을 했습니다.
  • 단, ComfirmationModal은 삭제 예정이라 리팩토링하지 않았습니다.

기타 작업 내용

  • Comment 컴포넌트는 CommentBottomSheet에서만 사용하기 때문에 폴더 위치를 옮겼습니다.
  • 바텀시트 데스크톱 UI 및 모달에서 닫기 버튼 관련 로직을 통일했습니다. 이때 XButton -> CloseButton으로 수정해 더 직관적인 컴포넌트 이름을 사용할 수 있도록 했습니다.
  • NavBar에 svg 컴포넌트를 적용하면서 이미지 크기가 예상했던 바와 달라 필요한 컴포넌트들에 대해 width, height props를 추가했습니다.

코드 리뷰 포인트

  • svg 파일에서 viewbox를 그대로 두고 width height만 수정하면, 원본 비율 그대로 크기만 수정하는 식으로 동작한다고 합니다! 따라서 특정 컴포넌트들에 대해 width, height props를 추가하고, 각각의 width, height를 width={width || 14}(이때 14는 기본으로 설정되어 있었던 값)와 같이 수정했습니다. 혹시 다른 컴포넌트들도 유사한 수정이 필요할 때 참고하시면 좋을 것 같습니다!

작업 화면

  • 없음

@gustn99 gustn99 added the refactor Improve code structure and readability label Dec 26, 2024
@gustn99 gustn99 requested a review from mimizae December 26, 2024 18:51
@gustn99 gustn99 self-assigned this Dec 26, 2024
@mimizae
Copy link
Collaborator

mimizae commented Dec 27, 2024

확인했습니답 신경 써주셔서 감사합니다 😋👏🏻👏🏻👏🏻 그냥 제가 공통 컴포넌트 리팩토링 165에서 모든 컴포넌트화 된 svg 아이콘들의 너비와 높이도 props로 받을 수 있게끔 수정할까용?? 언젠가 건들 수 있으니...

@gustn99
Copy link
Collaborator Author

gustn99 commented Dec 27, 2024

확인했습니답 신경 써주셔서 감사합니다 😋👏🏻👏🏻👏🏻 그냥 제가 공통 컴포넌트 리팩토링 165에서 모든 컴포넌트화 된 svg 아이콘들의 너비와 높이도 props로 받을 수 있게끔 수정할까용?? 언젠가 건들 수 있으니...

헉스. 머지해버려서 괜찮으시다면 제가 166번에서 처리하겠습니다!

@gustn99 gustn99 merged commit 937d340 into dev Dec 27, 2024
1 check passed
@gustn99 gustn99 deleted the feat/OD-166 branch December 27, 2024 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Improve code structure and readability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants