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

feat: Toast 구현 #18

Merged
merged 12 commits into from
Nov 7, 2023
Merged

feat: Toast 구현 #18

merged 12 commits into from
Nov 7, 2023

Conversation

nijuy
Copy link
Collaborator

@nijuy nijuy commented Oct 31, 2023

1️⃣ 어떤 작업을 했나요? (Summary)

기존 코드에 영향을 미치지 않는 변경사항

  • textBright 색상 추가 (Toast + 추후 Tooltip, Chip에서 사용)

  • <Toast> 컴포넌트 추가

  • 거북이 같은 속도로 useToast 훅 추가...🐢

2️⃣ 알아두시면 좋아요!

  • 애니메이션 속도와 timing-function은 노션 참고했습니다

  • 애니메이션 속도는 상수로 정의해놓고 썼습니다.
    FADE_DURATION은 Toast가 뜰 때 / 지워질 때 걸리는 시간이고, 이외는 Toast 유지 시간을 의미해요~!

  • 제가...... 이전 YDS에서 ToastMessage라는 이름으로 구현되어 있는 걸 구현 끝나고 알았습니다 ^^....
    일단은 Toast로 구현한 거 냅뒀는디 이 부분 이름 수정 필요하다 느끼시면 ToastMessage로 고칠게용~!

3️⃣ 추후 작업

  • PR 리뷰 기반 수정
  • 머지 후 노션에 Toast 문서 작성

4️⃣ 체크리스트 (Checklist)

src/hooks/useToast/useToast.ts Outdated Show resolved Hide resolved
src/hooks/useToast/useToast.ts Show resolved Hide resolved
- useToast에서 사용되지 않는 UseToastProps 타입과 관련 import문을 삭제했습니ㅏㄷ
- long Toast에서 FADE_DURATION을 사용하도록 수정했습니다
Copy link
Member

@Hanna922 Hanna922 left a comment

Choose a reason for hiding this comment

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

수고하셨습니당~!!!👍🏻👍🏻👍🏻 개인적으로 Toast 이름은 그대로 둬도 괜찮을 것 같아요!

@nijuy nijuy added the feat label Nov 7, 2023
@nijuy nijuy merged commit 8225033 into develop Nov 7, 2023
@nijuy nijuy deleted the feat/toast branch November 7, 2023 13:24
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.

2 participants