Skip to content

style(web): PC 공지 배너 콘텐츠 영역 내부로 이동#59

Merged
bbbang105 merged 1 commit intodevfrom
style/notice-banner-content-area
Mar 18, 2026
Merged

style(web): PC 공지 배너 콘텐츠 영역 내부로 이동#59
bbbang105 merged 1 commit intodevfrom
style/notice-banner-content-area

Conversation

@bbbang105
Copy link
Owner

Summary

  • PC에서 공지 배너가 사이드바+콘텐츠 전체 너비에 걸쳐 애매하게 표시되던 레이아웃을 수정
  • 태블릿/모바일과 동일하게 OVERVIEW 위 콘텐츠 영역 내부에 배치

Changes

파일 변경 내용
packages/web/src/components/layout/main-layout.tsx NoticeBannerMainLayout 루트에서 MainContent 내부로 이동, isAdmin prop 전달

Design Decisions

결정 이유
NoticeBannerMainContent 스크롤 영역 안에 배치 사이드바와 분리하여 콘텐츠 영역에만 표시, 모바일/태블릿 뷰와 일관성 유지
스크롤 컨테이너와 패딩 컨테이너 분리 배너는 콘텐츠 max-width와 무관하게 전체 너비, children은 기존 패딩 유지

Test Plan

  • PC(lg 이상)에서 공지 배너가 OVERVIEW 바로 위에 표시되는지 확인
  • 태블릿/모바일에서 기존과 동일하게 동작하는지 확인
  • 배너 접기/닫기 동작 정상 확인
  • 관리자 모드에서 배너 미표시 확인
  • 다크모드에서 배너 스타일 정상 확인

🤖 Generated with Claude Code

PC에서 공지 배너가 사이드바+콘텐츠 전체 너비에 걸쳐 애매하게 표시되던 문제 수정.
태블릿/모바일과 동일하게 OVERVIEW 위 콘텐츠 영역 내부에 배치.

Co-Authored-By: Claude <noreply@anthropic.com>
@bbbang105 bbbang105 requested a review from choihooo as a code owner March 18, 2026 06:41
@vercel
Copy link

vercel bot commented Mar 18, 2026

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

Project Deployment Actions Updated (UTC)
study-admin-web Ready Ready Preview, Comment Mar 18, 2026 6:42am

@bbbang105 bbbang105 added the 🌱 style 코드 의미에 영향을 주지 않는 변경사항 (코드 포맷팅, 오타 수정, 변수명 변경, 에셋 추가) label Mar 18, 2026
@bbbang105 bbbang105 merged commit 52e5023 into dev Mar 18, 2026
8 checks passed
@bbbang105 bbbang105 deleted the style/notice-banner-content-area branch March 18, 2026 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌱 style 코드 의미에 영향을 주지 않는 변경사항 (코드 포맷팅, 오타 수정, 변수명 변경, 에셋 추가)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant