Skip to content

Conversation

@huuitae
Copy link
Contributor

@huuitae huuitae commented Oct 16, 2025

📄 PR 내용 요약

알려진 페이지 모달 QA 이슈 수정했습니다.
에러 바운더리 추가했습니다.

✅ 작업 내용 상세

  1. 등록, 수정 모달 모바일 화면에서 ESC키 동작시 발생하는 문제
    <dialog> 태그의 기본 ESC 동작은 막고, router.back()을 통해 닫히는 건 가능합니다.

  2. 등록, 수정 모달 모바일 화면에서 버튼과 분리되는 문제
    기존에 있던 margin-top을 제거하고 height를 늘려 기존과 맞게 수정했습니다.

  3. Error Boundary
    기본적인 에러바운더리 세팅은 해둔 상태입니다.
    throw new Error("Test error for ErrorBoundary."); 이렇게 에러를 던지게 되면
    error_boundary_test

토스트 메시지로 우상단에 Error()안에 입력한 내용을 표출합니다.

에러 바운더리를 사용하기 위해선 Error가 생길만한 페이지의 layout.tsx파일에 <ErrorBoundary>컴포넌트로 감싸주어야 합니다.

export default function LoginLayout({ children }: { children: ReactNode }) {
  return (
      <ErrorBoundary>{children}</ErrorBoundary>
  );
}

📸 스크린샷 (선택사항)

💬 참고 사항

@huuitae huuitae requested review from junye0l, suuuuya and wlrnjs October 16, 2025 12:53
@huuitae huuitae self-assigned this Oct 16, 2025
@huuitae huuitae added 🐞 BugFix 제대로 작동하지 않음 🎨 Design UI/UX, 디자인 변경 labels Oct 16, 2025
@vercel
Copy link

vercel bot commented Oct 16, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
wine Ready Ready Preview Comment Oct 17, 2025 5:41am

@huuitae huuitae changed the title [#165] 페이지 모달 QA [#165] 등록/수정 모달 QA Oct 16, 2025
@huuitae huuitae changed the title [#165] 등록/수정 모달 QA [#165] 등록/수정 모달 QA, 에러 바운더리 추가 Oct 17, 2025
@huuitae huuitae merged commit 64cdc2b into develop Oct 17, 2025
3 checks passed
@huuitae huuitae deleted the feature/QA branch October 17, 2025 05:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 BugFix 제대로 작동하지 않음 🎨 Design UI/UX, 디자인 변경

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants