🔗 :: (#488) 로그인 버튼 Column 분리

[uson1004]

개요

로그인 페이지에서 키보드가 올라갔을 때 버튼이 화면 크기에 상관없이 보여지도록 개선

작업 내용

• 로그인 버튼 컬럼에서 분리

할 말

없음

Summary by CodeRabbit

릴리스 노트

• 스타일
  • 로그인 버튼을 화면 하단에 고정하고 키보드 등장 시 자동 여백(imePadding) 적용
  • 버튼을 메인 콘텐츠에서 분리해 하단 배치로 사용성 개선
  • 상단 바, 제목, 입력 필드, 비밀번호 재설정 버튼은 기존 위치 유지

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

SignInScreen 레이아웃이 변경되어 메인 콘텐츠는 세로 스크롤을 유지한 채 IME 및 내비게이션 바 패딩이 적용되고, 로그인 버튼이 화면 하단에 위치한 imePadding 적용 Box로 이동했습니다. 기존 상단 바, 제목, 입력 필드, 비밀번호 재설정 버튼 구조는 유지됩니다.

Changes

코호트 / 파일(s)	변경 요약
UI 레이아웃 변경 feature/signin/src/main/java/team/retum/signin/ui/SignInScreen.kt	Column에 verticalScroll 및 navigationBarsPadding 적용; 하단에 Box(fillMaxSize, imePadding, contentAlignment = BottomCenter) 추가하여 로그인 버튼을 Box 하단으로 이동; 관련 Box, imePadding, navigationBarsPadding, rememberScrollState, verticalScroll 임포트 추가 및 배치 변경

Estimated code review effort

🎯 2 (단순) | ⏱️ ~10분

• 확인할 점:
  • 추가된 임포트들이 실제로 사용되는지 검증
  • imePadding()과 navigationBarsPadding()의 상호작용으로 버튼/입력 필드가 가려지지 않는지 확인
  • 스크롤 동작 및 기존 상단 바/입력 필드 레이아웃 유지 여부 검증

시

🐰 아래에 버튼이 포근히 앉아,
스크롤은 부드럽게 춤추고,
시스템 바도 예의를 지키네.
입력은 제자리에, 화면은 여유롭게,
작은 터치로 큰 안정감을 선사해요 ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 주요 변경 사항을 명확하게 요약합니다. 로그인 버튼을 Column에서 분리하는 내용이 코드 변경사항과 일치합니다.
Description check	✅ Passed	PR 설명이 필수 템플릿 구조를 따르고 있으며, 개요, 작업 내용, 할 말 섹션이 모두 포함되어 있습니다.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ 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 refactor/488-로그인-컴포저블-끼리-부딫힐-시-깨짐

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 85550f5 and b969450.

📒 Files selected for processing (1)

• feature/signin/src/main/java/team/retum/signin/ui/SignInScreen.kt (2 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• feature/signin/src/main/java/team/retum/signin/ui/SignInScreen.kt

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: lint_and_build (build)

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

feature/signin/src/main/java/team/retum/signin/ui/SignInScreen.kt (1)

132-164: 로그인 버튼 분리 방향은 좋고, 네비게이션 바 인셋만 한 번 더 점검해 주세요

• Column에 verticalScroll(rememberScrollState())를 붙이고, 로그인 버튼을 별도 Box로 분리해 imePadding()으로 키보드 위에 고정한 구조는 PR 목적(키보드 올라왔을 때 버튼 항상 보이기)에 잘 맞습니다.
• 다만 하단 Box에는 imePadding()만 적용되어 있어서, 앱이 edge‑to‑edge로 네비게이션 바 영역까지 그리는 환경이라면 키보드가 내려간 상태에서 버튼이 시스템 네비게이션 바와 겹칠 수 있습니다.

두 가지를 제안드립니다.

1. 하단 Box에 네비게이션 바 인셋도 함께 적용

   Box(
       modifier = Modifier
           .fillMaxSize()
           .imePadding()
           .navigationBarsPadding(),
       contentAlignment = Alignment.BottomCenter,
   ) { ... }

   이렇게 하면 키보드가 없을 때는 네비게이션 바, 있을 때는 IME 높이를 고려해 항상 버튼이 안전 영역 안에 위치합니다.

2. 구조를 조금 더 명시적으로 만들기 (선택 사항)
   현재는 Column과 Box 두 개가 최상위에서 겹쳐지는데, 보기에는 괜찮지만 의도를 더 분명히 하려면 바깥에 Box를 두고 그 안에 Column과 하단 버튼을 같이 두는 패턴도 고려할 수 있습니다.

네비게이션 바와의 겹침 여부는 실제 기기(특히 3버튼 네비게이션 / 제스처 네비게이션 기기)에서 한 번만 확인 부탁드립니다.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ff02739 and 85550f5.

📒 Files selected for processing (1)

• feature/signin/src/main/java/team/retum/signin/ui/SignInScreen.kt (3 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: lint_and_build (build)

🔇 Additional comments (1)

feature/signin/src/main/java/team/retum/signin/ui/SignInScreen.kt (1)

5-17: 스크롤/인셋 처리를 위한 import 추가는 적절해 보입니다

Box, imePadding, navigationBarsPadding, rememberScrollState, verticalScroll 모두 아래에서 정상적으로 사용되고 있고, 불필요한 import 도 보이지 않습니다.