Skip to content

토스트 알림 위치 변경#359

Open
Seong-Myeong wants to merge 1 commit intomainfrom
feature/#354-edit-design-toast-location
Open

토스트 알림 위치 변경#359
Seong-Myeong wants to merge 1 commit intomainfrom
feature/#354-edit-design-toast-location

Conversation

@Seong-Myeong
Copy link
Contributor

관련 이슈

PR 설명

  • 기존 구현에 오른쪽 상단으로 고정되어있던 부분이 중앙 하단으로 변경되어 위치 변경

@Seong-Myeong Seong-Myeong self-assigned this Feb 9, 2026
@Seong-Myeong Seong-Myeong linked an issue Feb 9, 2026 that may be closed by this pull request
@coderabbitai
Copy link

coderabbitai bot commented Feb 9, 2026

개요

ToastContainer 컴포넌트에서 모달 스토어 의존성과 모달 관련 레이아웃 로직을 제거했습니다. 조건부로 동적인 스타일링 클래스를 정적 값으로 단순화하고, 모달 상태에 따른 조건부 래퍼와 정렬 로직을 삭제했습니다. 포탈 생성 및 토스트 렌더링 동작은 유지되며, 모달 상태와 관계없이 일관된 중앙 정렬 방식으로 토스트를 표시합니다.

관련 가능성 있는 PR

  • PR #276: 동일한 ToastContainer 컴포넌트에 대한 모달 스토어 의존성 제거 및 레이아웃 조건부 로직 단순화 작업과 직접 연관
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 변경 사항의 주요 내용과 정확히 일치합니다. 토스트 알림의 위치 변경이라는 핵심 변화를 명확하게 표현하고 있습니다.
Description check ✅ Passed PR 설명이 템플릿에 정의된 모든 필수 섹션을 포함하고 있으며, 관련 이슈와 변경 사항을 명확하게 기술하고 있습니다.
Linked Issues check ✅ Passed 코드 변경사항이 이슈 #354의 요구사항을 완벽하게 충족합니다. 토스트 알림을 오른쪽 상단에서 중앙 하단으로 이동시키는 목표가 구현되었습니다.
Out of Scope Changes check ✅ Passed 토스트 컨테이너 파일의 모든 변경사항이 토스트 위치 변경이라는 범위 내에 있으며, 무관한 수정사항은 발견되지 않습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/#354-edit-design-toast-location

No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Design] Toast 알림 위치 수정

2 participants