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

⚡ 공용 컴포넌트 리팩토링 #64

Merged
merged 8 commits into from
May 16, 2024

Conversation

BangDori
Copy link
Collaborator

@BangDori BangDori commented May 16, 2024

작업 이유


작업 사항

1️⃣ 토스트 컴포넌트

  • 토스트 컴포넌트의 적용은 최상단에서 이루어지지만, 토스트 컴포넌트를 사용하는 함수는 React Query의 onError 혹은 다른 컴포넌트들에서 사용될 수 있다고 생각하여 shared/toast 폴더로 이동하였습니다.
  • 다른 컴포넌트들에서 토스트 컴포넌트들을 사용할 경우를 고려하여 메시지와 아이콘을 매개변수로 받도록 수정하였습니다.

2️⃣ 캐러셀

  • 캐러셀 컴포넌트를 feed/widgets로 이동하였습니다.

3️⃣ 아이폰 레이아웃 수정

  • iPhone SE 사이즈가 현재 아이폰 크기의 75%와 가장 유사하다고 생각하여 아이폰 레이아웃 최소 및 시작 사이즈를 75%로 고정하였습니다.
  • 현재 아이패드를 고려하지 않기에 아이폰 레이아웃이 활성화되는 시점을 최소 너비 1080px로 설정하였습니다.

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

  • 토스트 컴포넌트의 위치가 적절한가요?
  • 토스트 컴포넌트의 사용성을 적절히 고려하였나요?

발견한 이슈

  • 없음

@BangDori BangDori requested a review from Legitgoons May 16, 2024 03:41
@BangDori BangDori self-assigned this May 16, 2024
Copy link

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

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

Copy link
Member

@Legitgoons Legitgoons left a comment

Choose a reason for hiding this comment

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

수고하셨습니다. 병합하셔도 좋습니다!

src/shared/toast/dir/toastHandlers.tsx Show resolved Hide resolved
@BangDori BangDori merged commit 4d980ed into main May 16, 2024
2 checks passed
@BangDori BangDori deleted the refactor/issue-60-shared-components branch May 16, 2024 05:44
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.

2 participants