🛠️ 공통 Button, Modal 컴포넌트 수정

[Legitgoons]

작업 이유

#61
기존 공통 Button, Modal 컴포넌트의 문제점 해결

작업 사항

1️⃣ ActiveButton / BasicButton 제거

• 해당 컴포넌트들의 사용성이 부족하여 제거하였습니다.

2️⃣ ConfirmModalButton 및 FollowButton 추가

• 기존의 Button 컴포넌트 대신, 공통으로 사용되는 해당 컴포넌트들을 구현하였습니다.
• 두 버튼 중, shared 외부에서도 사용되는 FollowButton만 index.ts에 추가하였습니다.

3️⃣ ConfirmModal 스타일에 분기문 정리

• isReport 분기문을 삭제하였습니다.

4️⃣ ConfirmReportModal 내부 버튼 ReportModalButton 구현

• 아래의 속성들을 props로 가지는 ReportModalButton 컴포넌트를 구현하였습니다.

  onClick: React.MouseEventHandler<HTMLButtonElement>;
  children: React.ReactNode;
  styleClass: ReportModalButtonStyle;
  type?: 'button' | 'submit' | 'reset'; // 기본 값 'submit'
  isDisabled?: boolean; // 기본 값 'false'

리뷰어가 중점적으로 확인해야 하는 부분

• ReportModalButton의 구현이 적절한지
• 추가적으로 수정해야 할 부분이 있는지
• 수정된 컴포넌트들의 사용성과 가독성이 향상되었는지

발견한 이슈

X

[aws-amplify-ap-northeast-2]

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-63.d37mn03xh3qyyz.amplifyapp.com

[BangDori]

FSD 아키텍처의 관점에 대해 이해하려고 노력하며 프로젝트를 진행하는 모습이 인상 깊네요 ㅎㅎ

이슈에 등록되어있었던 task는 모두 완료 처리하였습니다!

[BangDori]

:disabled 선택자를 활용하는 편이 더 낫다고 생각하는데 이에 대해 의견 공유 부탁드릴게요!

[BangDori]

고생하셨습니다!