[FIX] 폰트 사이즈 수정

[Sangyoon98]

📝 Summary

폰트가 너무 커서 수정

🙏 Question & PR point

📬 Reference

Summary by CodeRabbit

릴리스 노트

• Style (스타일)

  • 타이포그래피의 각 텍스트 스타일에 대해 폰트 크기를 명시적으로 설정하여 전체 UI의 일관성을 개선했습니다.
  • 로그인/회원가입 화면의 레이아웃 인셋 처리 방식을 업데이트하여 키보드 및 내비게이션 바와의 간격 관리를 개선했습니다.
  • 회원가입 화면의 제출 버튼에 내비게이션 바 패딩을 추가하여 여백을 최적화했습니다.

• Chores

  • 앱 버전을 1.0.0 → 1.0.1(버전코드 1 → 2)로 업데이트했습니다.

[coderabbitai]

Walkthrough

세 개의 UI 파일에서 타이포그래피와 스캐폴드 인셋/패딩을 조정했고, 앱 버전 정보를 app/build.gradle.kts에서 versionCode 1->2, versionName "1.0.0"->"1.0.1"로 업데이트했습니다.

Changes

Cohort / File(s)	Summary
타이포그래피 폰트 크기 설정 app/src/main/java/com/sampoom/android/core/ui/theme/Type.kt	Typography의 모든 스타일(display*, headline*, title*, body*, label*)에서 기존 fontFamily 복사 대신 fontFamily와 명시적 fontSize(sp)를 함께 설정하도록 변경
스캐폴드 콘텐츠 인셋 변경 app/src/main/java/com/sampoom/android/feature/user/ui/LoginScreen.kt, app/src/main/java/com/sampoom/android/feature/user/ui/SignUpScreen.kt	WindowInsets.ime 참조를 ScaffoldDefaults.contentWindowInsets로 교체하고 ScaffoldDefaults 임포트 추가
회원가입 화면 버튼 패딩 app/src/main/java/com/sampoom/android/feature/user/ui/SignUpScreen.kt	제출 버튼(modifier)에 .navigationBarsPadding() 추가; navigationBarsPadding 임포트 추가
버전 정보 업데이트 app/build.gradle.kts	android.defaultConfig의 versionCode를 1→2, versionName을 "1.0.0"→"1.0.1"로 변경

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• 주의 검토 항목:
  • Type.kt: 각 스타일에 지정된 fontSize 값이 디자인 사양과 일치하는지 확인
  • LoginScreen.kt / SignUpScreen.kt: ScaffoldDefaults.contentWindowInsets로 변경해도 키보드/인셋 동작이 의도대로 유지되는지 UI에서 확인
  • SignUpScreen.kt: .navigationBarsPadding() 적용으로 버튼 레이아웃/오버랩 여부 확인
  • app/build.gradle.kts: 버전 정보 변경이 배포 파이프라인 규칙에 부합하는지 확인

Poem

🐰 폰트가 크기를 찾았네, 스크린에 살며시 놓이고,
스캐폴드는 자리를 지키며, 인셋은 더 조용해졌네.
버튼은 바깥쪽을 존중하고, 버전은 한 걸음 더 나아가,
작은 깡충이 기쁨으로 깡충—변화는 또 하나의 도약! 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	PR 제목인 "[FIX] 폰트 사이즈 수정"은 Type.kt 파일의 주요 변경사항을 명확하게 설명하고 있습니다. 타이포그래피의 모든 스타일에 명시적인 폰트 사이즈를 추가하여 각 스타일별로 지정된 크기(예: displayLarge는 43.sp)를 적용한 것이 주요 변경이며, 이는 PR 설명의 의도인 "폰트가 너무 커서 수정"과 일치합니다. 제목이 PR의 핵심 목표와 주요 변경사항을 적절히 전달하고 있으며, 개발자가 히스토리를 스캔할 때 폰트 크기 조정이 주요 내용임을 바로 이해할 수 있습니다.

✨ 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 SPM-270

---

📜 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 7620a62 and aa8adaa.

📒 Files selected for processing (1)

• app/build.gradle.kts (1 hunks)

🔇 Additional comments (1)

app/build.gradle.kts (1)

35-36: 버전 번호 업데이트는 패치 릴리스에 적절합니다.

versionCode를 1에서 2로, versionName을 "1.0.0"에서 "1.0.1"로 업데이트한 것은 버그 수정에 대한 표준 패치 버전 관례를 따릅니다. 버전 관리 측면에서는 문제가 없습니다.

다만, PR 설명과 댓글에서 저자가 기기에서 직접 테스트하지 않으셨다고 답변하신 점이 우려됩니다. 폰트 사이즈는 시각적 변경사항이므로 실제 기기(또는 에뮬레이터)에서 검증이 필수적입니다. 이 파일만으로는 UI 파일들(Type.kt, LoginScreen.kt, SignUpScreen.kt)의 실제 폰트 크기 변경 사항을 확인할 수 없으므로, 해당 파일들에 대한 검토 및 테스트 검증이 필요합니다.

---

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.}

[taemin3]

확인했습니다

[Lee-Jong-Jin]

테스트 해봤는데 엄청 크긴 했음

[CHOOSLA]

제 기기에서 테스트하고 변경하신 건가요

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

app/src/main/java/com/sampoom/android/core/ui/theme/Type.kt (1)

7-7: 미사용 import를 제거해주세요.

dp import가 추가되었지만 이 파일에서 사용되지 않습니다.

-import androidx.compose.ui.unit.dp
 import androidx.compose.ui.unit.sp

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 16fb117 and 7620a62.

📒 Files selected for processing (3)

• app/src/main/java/com/sampoom/android/core/ui/theme/Type.kt (2 hunks)
• app/src/main/java/com/sampoom/android/feature/user/ui/LoginScreen.kt (2 hunks)
• app/src/main/java/com/sampoom/android/feature/user/ui/SignUpScreen.kt (3 hunks)

🔇 Additional comments (5)

app/src/main/java/com/sampoom/android/core/ui/theme/Type.kt (1)

56-66: 본문 텍스트 폰트 크기가 너무 작을 수 있습니다.

bodyLarge(12sp), bodyMedium(12sp), bodySmall(10sp)의 폰트 크기가 Material Design 3 기본값(16sp, 14sp, 12sp)보다 상당히 작습니다. 특히 bodyLarge와 bodyMedium이 동일한 크기(12sp)를 사용하고 있어 계층 구조가 명확하지 않습니다.

실제 기기에서 가독성을 테스트하여 사용자 경험에 문제가 없는지 확인해주세요.

app/src/main/java/com/sampoom/android/feature/user/ui/SignUpScreen.kt (2)

15-15: ScaffoldDefaults.contentWindowInsets 사용이 적절합니다.

WindowInsets.ime에서 ScaffoldDefaults.contentWindowInsets로 변경하여 키보드뿐만 아니라 네비게이션 바 등 다양한 시스템 인셋을 자동으로 처리할 수 있습니다.

Also applies to: 23-23, 89-89

---

200-200: 네비게이션 바 패딩 중복 가능성을 확인해주세요.

ScaffoldDefaults.contentWindowInsets가 이미 네비게이션 바 인셋을 처리하고 있을 수 있어, .navigationBarsPadding()을 추가하면 패딩이 중복 적용될 수 있습니다.

실제 기기에서 버튼 하단에 불필요한 여백이 생기지 않는지 확인해주세요.

app/src/main/java/com/sampoom/android/feature/user/ui/LoginScreen.kt (2)

17-17: ScaffoldDefaults.contentWindowInsets 사용이 적절합니다.

WindowInsets.ime에서 ScaffoldDefaults.contentWindowInsets로 변경하여 시스템 인셋 처리가 개선되었습니다.

Also applies to: 71-71

---

85-93: imePadding()이 중복될 수 있습니다.

88번 줄의 .imePadding()이 ScaffoldDefaults.contentWindowInsets와 중복으로 키보드 패딩을 처리할 수 있습니다. 또한 SignUpScreen에서는 Submit 버튼에 .navigationBarsPadding()을 추가했지만, 이 화면에서는 추가하지 않아 일관성이 부족합니다.

두 화면에서 패딩 처리 방식을 통일하고, 불필요한 중복 패딩을 제거하는 것을 검토해주세요.

[Sangyoon98]

제 기기에서 테스트하고 변경하신 건가요

아닙니다^^