Skip to content

Refactor/answer 개선#66

Merged
ToKyun02 merged 7 commits intomergefrom
refactor/Answer_개선
Dec 25, 2024

Hidden character warning

The head ref may contain hidden characters: "refactor/Answer_\uac1c\uc120"
Merged

Refactor/answer 개선#66
ToKyun02 merged 7 commits intomergefrom
refactor/Answer_개선

Conversation

@junAlexx
Copy link
Contributor

@junAlexx junAlexx commented Dec 25, 2024

✍️ Description

Answer 페이지 개선사항에 관한 PR 입니다.

수정사항

  • 케밥 메뉴 토글 기능 추가 및 수정 중 '수정하기' 버튼 비활성화
  • 삭제 동작 시 사용자 확인용 모달 추가
  • 케밥 메뉴 로딩처리(로딩 시 비활성화) 및 삭제 확인 모달 애니메이션 추가(버튼 hover시 scale-105로)
  • 사용자 동작에 따른 토스트 메세지 구현
  • 수정 취소 버튼 구현
  • 모달 슬라이드업 애니메이션 추가

사용자 확인 모달

image

입력폼 우측 상단 수정 취소 버튼

image

사용자 동작에 따른 토스트 메세지

image

✅ Checklist

PR

  • Branch Convention 확인

    epic/ 에픽, feat/ 피쳐, fix/ 버그 수정, refactor/ 개선

  • Base Branch 확인
  • 적절한 Label 지정
  • Assignee 및 Reviewer 지정

Test

  • 로컬 작동 확인

Additional Notes

  • (없음)

@junAlexx junAlexx requested a review from ToKyun02 December 25, 2024 04:19
@junAlexx junAlexx self-assigned this Dec 25, 2024
@netlify
Copy link

netlify bot commented Dec 25, 2024

Deploy Preview for team6openmind ready!

Name Link
🔨 Latest commit 8581e97
🔍 Latest deploy log https://app.netlify.com/sites/team6openmind/deploys/676b8e3c4f7b720008eedc78
😎 Deploy Preview https://deploy-preview-66--team6openmind.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@ToKyun02 ToKyun02 left a comment

Choose a reason for hiding this comment

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

로컬에서 동작확인했습니다!

오류가 발생할 코드는 없고, 코드 쪽에서 참고하시면 좋을 것 같은 건 코멘트 남겼으니 확인하시면 될 것 같습니다!

고생하셨습니다😊

@ToKyun02 ToKyun02 merged commit 66f0e82 into merge Dec 25, 2024
5 checks passed
@junAlexx
Copy link
Contributor Author

로컬에서 동작확인했습니다!

오류가 발생할 코드는 없고, 코드 쪽에서 참고하시면 좋을 것 같은 건 코멘트 남겼으니 확인하시면 될 것 같습니다!

고생하셨습니다😊

팀장님이 가장 고생 많으셨죠 :) 피드백 감사합니다!!

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