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] 북마크 이동 시 토스트 알람 생성 #891

Merged
merged 2 commits into from
Jan 1, 2025

Conversation

dmdgpdi
Copy link
Collaborator

@dmdgpdi dmdgpdi commented Dec 30, 2024

What is this PR? 🔍

Changes 📝

ScreenShot 📷

_2024_12_30_21_22_29_340.mp4

Precaution

@dmdgpdi dmdgpdi added feat 기능 frontend 프론트 관련 labels Dec 30, 2024
@dmdgpdi dmdgpdi self-assigned this Dec 30, 2024
@dmdgpdi dmdgpdi requested a review from kimminkyeu December 30, 2024 12:27
@dmdgpdi dmdgpdi added the D-1 label Dec 30, 2024
@kimminkyeu
Copy link
Collaborator

LGTM. 위치를 왜 변경하셨는지 궁금해요! 토스트 메시지를 화면에서 최대한 숨기기 위함인가요?

@dmdgpdi
Copy link
Collaborator Author

dmdgpdi commented Dec 30, 2024

LGTM. 위치를 왜 변경하셨는지 궁금해요! 토스트 메시지를 화면에서 최대한 숨기기 위함인가요?

@kimminkyeu
상단 가운데로 다시 돌려놓을까요? 우측 상단도 좋을 것 같습니다!
다른 이유는 없고, 드래그 앤 드랍한 이후, 조금 더 늦게 토스트 알람이 뜹니다.(성공했을 때만 뜨게 했습니다.) 그래서 가운데에 있으면 사용자가 성공 토스트 알람을 기다리게 될 것 같아서 우측 하단으로 두었습니다!

@kimminkyeu
Copy link
Collaborator

넵 우측 상단 좋다고 생각합니다!.
제 기준에선 우측 하단은 뭐가 떠도 눈에 보이지 않아서, 가운데는 너무 잘 보이니 우측 상단이 적당할 것 같네요~!

@dmdgpdi
Copy link
Collaborator Author

dmdgpdi commented Dec 30, 2024

넵 우측 상단 좋다고 생각합니다!. 제 기준에선 우측 하단은 뭐가 떠도 눈에 보이지 않아서, 가운데는 너무 잘 보이니 우측 상단이 적당할 것 같네요~!

넵 우측 상단 좋다고 생각합니다!. 제 기준에선 우측 하단은 뭐가 떠도 눈에 보이지 않아서, 가운데는 너무 잘 보이니 우측 상단이 적당할 것 같네요~!

_2024_12_31_08_51_46_351.mp4
_2024_12_31_08_52_51_26.mp4

제가 다시 보니 우측 상단보다 정 가운데가 더 좋아보여서 옮겼습니다! 혹시 가운데로 옮겨도 괜찮으신가요? 이상하면 다시 옮기겠습니다!

@kimminkyeu
Copy link
Collaborator

@dmdgpdi 넵 저도 가운데가 좋은 것 같아요! 가운데로 픽스하시죠

@dmdgpdi dmdgpdi merged commit b68f248 into fe-develop Jan 1, 2025
1 check passed
@dmdgpdi dmdgpdi deleted the front/feat/#816 branch January 1, 2025 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-1 feat 기능 frontend 프론트 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants