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

🔨 refactor: toast UI 수정 #290

Merged
merged 4 commits into from
Apr 18, 2024
Merged

🔨 refactor: toast UI 수정 #290

merged 4 commits into from
Apr 18, 2024

Conversation

easyhyun00
Copy link
Collaborator

🧩 이슈 번호

✅ 작업 사항

  1. toast 가 사라지는 시간을 통일시키기 위해 autoClose 를 1500 으로 설정했습니다.

  2. 사파리에서 이미지 파일 저장 시, 사진이 저장 되기 전 토스트가 나타나는 현상이 있고,
    웬만한 모든 브라우저에서는 파일이 저장 되었다는 것을 브라우저에서 보여주기 때문에 해당 토스트를 제거하였습니다.

[기존]

  1. 이미지 삭제 토스트 success 로 변경하였습니다.
image

👩‍💻 공유 포인트 및 논의 사항

+) 오늘 테스트하다가 발견한 사실인데 카카오 인앱브라우저에서는 이미지를 다운로드 하는 게 안된다고 하네요...
그래서 강제로 외부 브라우저로 이동해야 될 것 같다고 생각이 드네요.

참고 https://devtalk.kakao.com/t/topic/128291

@easyhyun00 easyhyun00 added the 🔨 Refactor 리팩토링 작업 label Apr 17, 2024
@easyhyun00 easyhyun00 self-assigned this Apr 17, 2024
@easyhyun00 easyhyun00 requested a review from bbearcookie as a code owner April 17, 2024 11:21
@easyhyun00 easyhyun00 changed the title 🔨 refactor: toast UI 수정 🔨 refactor: toast UI 수정 Apr 17, 2024
Copy link

github-actions bot commented Apr 17, 2024

🚀 Storybook Preview 보러가기: https://65a6c73d536a3c43b7c5c9bb-coxnkdclge.chromatic.com/
⏰ 배포 일시: Thu Apr 18 2024 13:51:20 GMT+0900

Copy link

github-actions bot commented Apr 17, 2024

🎮 React Preview 보러가기: https://dnd-10th-4-frontend-h0gco72bv-bbearcookies-projects.vercel.app
⏰ 배포 일시: Thu Apr 18 2024 13:51:37 GMT+0900

Copy link
Member

@bbearcookie bbearcookie left a comment

Choose a reason for hiding this comment

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

수고하셨어요!!
카카오톡 인앱 브라우저에서 파일 다운로드가 안되는 문제가 있었군요 😂

찾아보니 앱마다 탈출 방법이 조금씩 다른 것 같기도 하네요..
https://burndogfather.com/271

@@ -18,6 +18,7 @@ const AdminApp = () => {
</ApiErrorBoundary>
</UnknownErrorBoundary>
<ToastContainer
autoClose={1500}
Copy link
Member

Choose a reason for hiding this comment

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

오호! 토스트 자동 닫기 시간이 전역으로 지정되었군요 👍😊

@easyhyun00 easyhyun00 merged commit 7860a72 into main Apr 18, 2024
7 checks passed
@easyhyun00 easyhyun00 deleted the refactor/#289/toast-ui branch April 18, 2024 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Refactor 리팩토링 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

토스트 UI 수정
2 participants