Skip to content

fix: 모바일 시뮬레이터 하단 safe-area 여백을 보장한다 - #273

Merged
yereong merged 1 commit into
mainfrom
cha-169-fix-mobile-simulator-safe-area
Aug 22, 2026
Merged

fix: 모바일 시뮬레이터 하단 safe-area 여백을 보장한다#273
yereong merged 1 commit into
mainfrom
cha-169-fix-mobile-simulator-safe-area

Conversation

@yereong

@yereong yereong commented Aug 22, 2026

Copy link
Copy Markdown
Member

무엇을 변경했나요?

모바일 시뮬레이터 콘텐츠 최하단에 safe-area를 포함한 명시적 spacer를 추가했습니다.

왜 변경했나요?

기존 하단 padding이 flex 스크롤 컨테이너에서 콘텐츠 끝 여백으로 보장되지 않아, 모바일 최하단에서 계산 안내 카드가 화면에 붙는 문제가 있었습니다. 로그인 후 채널 선택 상태의 필터 버튼 회피 여백은 유지했습니다.

Linear: https://linear.app/chaesozip/issue/CHA-169/bug-%EC%8B%9C%EB%AE%AC%EB%A0%88%EC%9D%B4%ED%84%B0-%ED%8E%98%EC%9D%B4%EC%A7%80-%EB%AA%A8%EB%B0%94%EC%9D%BC-%EB%A7%A8%EC%95%84%EB%9E%98%EC%97%90%EC%84%9C-safe-area

Closes CHA-169

확인 사항

필수 확인

  • node --run build 통과 — 로컬 개발 서버가 공유 중인 .next 캐시와 충돌해 완료하지 못함
  • node --run lint 통과
  • 관련 테스트 확인 — 시뮬레이터 페이지 테스트 2건 통과
  • 관련 이슈를 연결했습니다. Closes CHA-169

기능 검증

  • 변경된 기능을 모바일 375×667 환경에서 직접 확인했습니다
  • 사용자 플로우가 의도대로 동작하는지 확인했습니다
  • API 연동 또는 상태 관리 변경 사항을 확인했습니다 — 해당 없음
  • 에러/로딩/빈 상태 등 예외 케이스를 확인했습니다 — 해당 없음
  • 변경된 동작에 대한 테스트를 추가하거나 업데이트했습니다

설정 및 문서

  • 사용자에게 노출되는 문서를 업데이트했습니다 — 해당 없음
  • 새로운 의존성이나 설정 변경 사항을 문서화했습니다 — 해당 없음
  • 환경 변수 추가/변경 사항을 명확히 설명했습니다 — 해당 없음
  • 민감한 값, 토큰, 인증 정보가 포함되지 않았습니다

해당되는 경우

  • 보안 영향을 검토했습니다 — 해당 없음
  • 성능 영향을 검토했습니다 — 레이아웃 spacer만 추가
  • 기존 동작을 변경하는 내용은 PR 설명에 명확히 작성했습니다
  • 스크린샷 또는 화면 녹화를 첨부했습니다 — 미첨부

@linear-code

linear-code Bot commented Aug 22, 2026

Copy link
Copy Markdown

CHA-169

@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: d213971d-03ec-4d6b-91db-098944b94bd7


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.

@yereong
yereong marked this pull request as ready for review August 22, 2026 03:53
@github-actions
github-actions Bot requested a review from DongjaJ August 22, 2026 03:53
@codecov

codecov Bot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yereong
yereong merged commit 66abc82 into main Aug 22, 2026
9 checks passed
@yereong
yereong deleted the cha-169-fix-mobile-simulator-safe-area branch August 22, 2026 04:11
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