[Refactor] Scaffold 공통 + 스크롤 #530#531
Merged
Merged
Conversation
- UndabangScaffold 신규 (presentation/compose/components/layout/Scaffold.kt)
- Material3 Scaffold 기반, containerColor 기본값 ColorWhite300
- topBar / bottomBar 슬롯 + 시스템 바·키보드 인셋 자동 분배
- verticalScroll 누락 6개 화면에 적용
- SettingScreen, NotificationScreen
- Column에 verticalScroll 단순 추가
- PermissionScreen, MatchingGuideScreen, ExercisePlaceRegisterScreen
- 하단 PrimaryButton을 bottomBar 슬롯으로 이동
- 본문 Column에 verticalScroll
- TermsScreen
- 약관 두 줄 + 확인 버튼을 bottomBar 슬롯으로 묶음 (작은 디바이스에서 약관·버튼이 화면 밖으로 잘리지 않게)
- 본문은 타이틀만
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- 같은 SettingMenuItem 호출 8번을 SettingMenuItemData 리스트로 묶어 LazyColumn으로 그림 - 기존 항목별 파라미터(iconRes/labelRes/onClick/trailingText/showDivider)는 data class로 보존 - 셀프 코드리뷰 반영 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
410413e to
13b4d01
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#️⃣연관된 이슈
#530
📝작업 내용
공통 화면 골격 컴포넌트를 도입하고, 스크롤 누락 6개 화면에 적용했습니다
presentation/.../components/layout/Scaffold.kt)기존 Compose 화면 전체를 Scaffold로 일괄 전환하는 작업은 후속 PR로 분리했습니다
스크린샷 (선택)
💬리뷰 요구사항(선택)