Skip to content

fix: 레이아웃 수정#55

Merged
haeni82 merged 1 commit into
developfrom
feature/policy-api
Mar 16, 2026
Merged

fix: 레이아웃 수정#55
haeni82 merged 1 commit into
developfrom
feature/policy-api

Conversation

@haeni82
Copy link
Copy Markdown
Contributor

@haeni82 haeni82 commented Mar 16, 2026

이슈

  • closed #이슈번호

✔️ 체크리스트

  • : Merge할 브랜치를 확인해 주세요.

🔍 작업 내용

⚠️ 주의 사항 / 기타

Summary by CodeRabbit

릴리스 노트

  • 새 기능

    • 알람 항목에 키보드 접근성 지원 추가
    • 로그 항목이 없을 때 빈 상태 메시지 표시
  • 개선사항

    • 전체 레이아웃 구조 및 스크롤 동작 개선
    • 멤버 선택 인터페이스를 아바타 기반으로 변경
    • 데이터 양도 모달 플로우 간소화
    • 각 페이지의 여백 및 간격 조정

@haeni82 haeni82 merged commit 283a51f into develop Mar 16, 2026
1 of 2 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 16, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 09c3db2a-a163-43fd-92ac-8867e6bc0901

📥 Commits

Reviewing files that changed from the base of the PR and between a56827b and ecbd060.

📒 Files selected for processing (10)
  • src/layout/Layout.tsx
  • src/page/Alarm.tsx
  • src/page/Detail/DetailPage.tsx
  • src/page/Log/LogPage.tsx
  • src/page/Main/MainPage.tsx
  • src/page/Policy/Policy.tsx
  • src/page/PolicyDetail/PolicyDetail.tsx
  • src/page/Setting/Setting.tsx
  • src/page/Support/Support.tsx
  • src/routes/Router.tsx

📝 Walkthrough

Walkthrough

레이아웃 기반 구조를 재설계하여 고정 전체화면 컨테이너와 전용 헤더/본문/하단 영역으로 개선했습니다. 이와 함께 여러 페이지 컴포넌트의 래퍼 컨테이너를 단순화하고, 라우터에서 BottomBar를 제거했습니다.

Changes

Cohort / File(s) Summary
Layout Infrastructure
src/layout/Layout.tsx
고정 전체화면 컨테이너(h-[100dvh], overflow-hidden)로 레이아웃을 재구성. StatusBar와 Header를 포함한 전용 상단 섹션, 스크롤 가능한 본문 영역(overflow-y-auto, min-h-0), 고정 하단 BottomBar 추가. 배경 처리를 단순화하고 내부 스크롤 동작을 개선.
Page Container Simplification
src/page/Detail/DetailPage.tsx, src/page/Setting/Setting.tsx, src/page/Support/Support.tsx, src/page/Main/MainPage.tsx
기존 relative overflow-y-auto mb-[60px] 래퍼를 제거하고 단순화된 flex 컨테이너로 통합. 하단 여백(pb-[60px]) 유지하면서 구조적 복잡도 감소.
Log & Detail Page Layout
src/page/Log/LogPage.tsx
외부 스크롤 컨테이너 제거 및 섹션 재정렬. 요약 카드 내 진행바 구조 단순화(getRemainingPercent 사용). 월별 로그 목록과 무한 스크롤 트리거 재배치. 로그 없음 상태에 대한 명시적 빈 상태 블록 추가.
Alarm & Policy Features
src/page/Alarm.tsx
카테고리 탭을 라벨에서 직접 렌더링하도록 변경. "전체 읽음" 버튼을 헤더에서 전용 컨트롤 영역으로 이동. 각 알람 항목에 접근성 지원 추가(onKeyDown 핸들러). NEW/READ 표시기와 메시지 블록 레이아웃 분리.
Policy & PolicyDetail Pages
src/page/Policy/Policy.tsx, src/page/PolicyDetail/PolicyDetail.tsx
Policy: 외부 컨테이너 제거 및 모달을 createPortal 기반으로 통합. 양도 요청 시 대상 선택 및 텍스트 확인 입력 추가. PolicyDetail: 탭 기반 네비게이션에서 아바타 기반 멤버 선택으로 변경. 기존 탭 구조(BlockTab, LimitTab) 유지하면서 상단 멤버 선택 영역 재구성.
Router Configuration
src/routes/Router.tsx
사용자 라우터 레이아웃에서 BottomBar 컴포넌트 및 import 제거. 어드민 라우팅 및 다른 로직은 변경 없음.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/policy-api
📝 Coding Plan
  • Generate coding plan for human review comments

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.

@coderabbitai coderabbitai Bot mentioned this pull request Mar 18, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant