fix: 감상문 가이드 선택 후 문장 맨 앞으로 커서 이동이 안되는 문제 해결#128
Merged
seoyoon513 merged 2 commits intodevelopfrom Aug 9, 2025
Merged
Conversation
Walkthrough
Changes
Sequence Diagram(s)(변경이 UI 컨테이너 교체 및 함수명 변경에 국한되어 있어 별도의 시퀀스 다이어그램은 생략합니다.) Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Assessment against linked issues
Assessment against linked issues: Out-of-scope changes(해당 변경사항 중 범위를 벗어난 변경사항은 없습니다.) Possibly related issues
Poem
Note 🔌 MCP (Model Context Protocol) integration is now available in Early Access!Pro users can now connect to remote MCP servers under the Integrations page to get reviews and chat conversations that understand additional development context. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ 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)
✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🔭 Outside diff range comments (1)
feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterUi.kt (1)
55-56: 내부 요소에 외부 modifier 재사용 지양 (Modifier vs modifier)외부에서 전달된 modifier는 루트에만 적용하는 것이 관례입니다. 내부 컴포넌트(여기서는 ProgressBar)에 같은 modifier를 재사용하면 상위에서 전달된 제약/클릭 영역 등이 예기치 않게 중복 적용될 수 있습니다.
- modifier = modifier.padding(horizontal = ReedTheme.spacing.spacing5), + modifier = Modifier.padding(horizontal = ReedTheme.spacing.spacing5),
🧹 Nitpick comments (3)
feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterUi.kt (3)
46-48: Scaffold → FullScreen 전환으로 innerPadding이 사라졌습니다. 시스템바/IME 인셋 보강 제안키보드/네비게이션바에 의한 하단 버튼 가림을 방지하려면 인셋 보정이 필요합니다. 컬럼 전체에 적용하는 방식을 제안드립니다(대안은 하단 버튼에만 적용, 아래 별도 코멘트 참고).
- ReedFullScreen(modifier = modifier.fillMaxSize()) { - Column(modifier = Modifier.fillMaxSize()) { + ReedFullScreen(modifier = modifier.fillMaxSize()) { + Column( + modifier = Modifier + .fillMaxSize() + .imePadding() + .navigationBarsPadding() + ) {추가로, 루트/자식에 동시로 fillMaxSize를 적용하는 것은 동작상 문제는 없지만, 루트에는 전달받은 modifier만 두고 자식에서 사이즈를 채우는 쪽이 일반적입니다.
- ReedFullScreen(modifier = modifier.fillMaxSize()) { + ReedFullScreen(modifier = modifier) {필요한 import:
import androidx.compose.foundation.layout.imePadding import androidx.compose.foundation.layout.navigationBarsPadding키보드 열림/닫힘, 가로모드, 제스처/3-버튼 내비 각각에서 하단 버튼 노출이 안정적인지 확인 부탁드립니다.
72-85: 대안: 인셋을 하단 버튼에만 적용하여 레이아웃 이동 최소화컬럼 전체가 아니라 하단 CTA에만 인셋을 적용하면 키보드 열림 시 레이아웃 점프를 줄일 수 있습니다. (위 컬럼 적용안과 중복 적용하지 마세요. 둘 중 하나만 선택)
- modifier = Modifier - .fillMaxWidth() - .padding(horizontal = ReedTheme.spacing.spacing5), + modifier = Modifier + .fillMaxWidth() + .padding(horizontal = ReedTheme.spacing.spacing5) + .navigationBarsPadding() + .imePadding(),필요한 import:
import androidx.compose.foundation.layout.imePadding import androidx.compose.foundation.layout.navigationBarsPadding
117-121: 접근성(a11y) 개선: contentDescription을 stringResource로 관리 권장하드코딩 문자열 대신 리소스로 관리하면 현지화와 톤/매너 일관성이 좋아집니다.
- contentDescription = "Record Complete Image", + contentDescription = stringResource(R.string.record_saved_dialog_image_content_description),리소스 추가 예:
<!-- feature/record/src/main/res/values/strings.xml --> <string name="record_saved_dialog_image_content_description">감상문 저장 완료 이미지</string>
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterUi.kt(5 hunks)
🧰 Additional context used
🧠 Learnings (8)
📓 Common learnings
Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#46
File: feature/search/src/main/kotlin/com/ninecraft/booket/feature/search/component/InfiniteLazyColumn.kt:83-95
Timestamp: 2025-07-14T00:46:03.843Z
Learning: seoyoon513과 팀은 한국어 주석을 선호하며, 한국어 주석을 영어로 번역하라는 제안을 하지 않아야 함
Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#75
File: feature/home/src/main/kotlin/com/ninecraft/booket/feature/home/HomeUi.kt:0-0
Timestamp: 2025-07-29T06:07:11.727Z
Learning: seoyoon513 팀에서는 UI 구현 단계에서 더미 데이터를 하드코딩하여 화면을 먼저 구현하고, 이후 서버 연동 시점에 실제 데이터로 교체하는 개발 방식을 사용합니다.
📚 Learning: 2025-07-20T12:34:23.786Z
Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#61
File: feature/webview/build.gradle.kts:17-21
Timestamp: 2025-07-20T12:34:23.786Z
Learning: Reed-Android 프로젝트에서는 `booket.android.feature` convention plugin을 사용하여 feature 모듈들의 공통 의존성을 관리한다. 이 plugin은 Circuit, Compose, 그리고 core 모듈들의 의존성을 자동으로 포함하므로, 각 feature 모듈의 build.gradle.kts에서는 특별한 의존성(예: libs.logger, libs.kakao.auth)만 별도로 선언하면 된다.
Applied to files:
feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterUi.kt
📚 Learning: 2025-07-31T23:22:02.816Z
Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#88
File: feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordsCollection.kt:25-59
Timestamp: 2025-07-31T23:22:02.816Z
Learning: BookDetailScreen의 RecordsCollection 컴포넌트는 상위 Column의 verticalScroll과 LazyColumn의 무한 스크롤을 동시에 지원해야 하는 중첩 스크롤 시나리오입니다. LazyColumn에 고정 높이를 설정하지 않으면 앱이 충돌하는 이슈가 있어서, 현재는 높이를 계산하여 설정하고 userScrollEnabled = false로 구현되어 있습니다. 향후 InfiniteLazyColumn과 nestedScrollConnection을 도입하여 전체 화면 스크롤 + LazyColumn 무한 스크롤을 지원할 예정입니다.
Applied to files:
feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterUi.kt
📚 Learning: 2025-07-08T12:33:01.863Z
Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#32
File: core/designsystem/src/main/kotlin/com/ninecraft/booket/core/designsystem/component/button/ButtonColorStyle.kt:10-16
Timestamp: 2025-07-08T12:33:01.863Z
Learning: Reed Android 프로젝트에서 KAKAO 버튼 스타일은 디자이너가 pressed 상태 색상을 별도로 정의하지 않았기 때문에 pressed 상태에서도 동일한 Kakao 색상을 사용한다.
Applied to files:
feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterUi.kt
📚 Learning: 2025-07-31T23:17:40.054Z
Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#88
File: feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordItem.kt:29-37
Timestamp: 2025-07-31T23:17:40.054Z
Learning: Reed-Android 프로젝트에서는 API가 준비되지 않은 상황에서 UI를 먼저 구현하고, API 연동 시점에 하드코딩된 데이터를 실제 데이터로 교체하는 개발 방식을 사용한다. RecordItem 컴포넌트의 emotionTags 매개변수도 API 연동 시점에 `text = emotionTags.joinToString(separator = "·") { "#$it" }`로 적용될 예정이다.
Applied to files:
feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterUi.kt
📚 Learning: 2025-07-08T03:21:24.693Z
Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#32
File: core/designsystem/src/main/kotlin/com/ninecraft/booket/core/designsystem/theme/Theme.kt:15-21
Timestamp: 2025-07-08T03:21:24.693Z
Learning: Reed Android 프로젝트에서는 다크모드/라이트모드 대응 계획이 없고 기본 테마 값만 사용할 예정이다.
Applied to files:
feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterUi.kt
📚 Learning: 2025-07-12T01:33:57.101Z
Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#45
File: core/designsystem/src/main/kotlin/com/ninecraft/booket/core/designsystem/component/appbar/ReedTopAppBar.kt:65-65
Timestamp: 2025-07-12T01:33:57.101Z
Learning: Reed Android 프로젝트에서 타이포그래피 사용 규칙: 톱 앱바(Top App Bar)에서는 `headline2SemiBold`를 사용하고, 바텀시트(Bottom Sheet)에서는 `heading2SemiBold`를 사용한다. 이는 의도적인 디자인 시스템 차별화이다.
Applied to files:
feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterUi.kt
📚 Learning: 2025-07-31T23:30:37.547Z
Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#88
File: feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/record/RecordDetailPresenter.kt:18-47
Timestamp: 2025-07-31T23:30:37.547Z
Learning: In Circuit architecture, presenters receive the Screen object directly as a constructor parameter (e.g., Assisted private val screen: RecordDetailScreen), and screen parameters are accessed through this screen object (e.g., screen.recordId). Screen parameters should not be added as separate constructor parameters.
Applied to files:
feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterUi.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: ci-build
🔇 Additional comments (4)
feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterUi.kt (4)
5-5: import 정리 LGTM필요한 Column import 추가 확인했습니다.
131-136: 미리보기 함수 업데이트 LGTM미리보기에서 새 컴포저블명을 사용하도록 일관성 있게 반영되었습니다.
36-39: RecordRegister 함수명 변경에 따른 호출부 확인 완료
전체 코드베이스에서RecordRegister(호출부를 검색한 결과 잔존 호출이 없습니다. 추가 조치는 필요하지 않습니다.
25-25: ReedFullScreen 전환 시 시스템바·IME 인셋 처리 확인 필요
ReedFullScreen에서는 내부적으로.systemBarsPadding()을 기본 적용하기 때문에, 기존 ReedScaffold의innerPadding처리 방식과 차이가 발생할 수 있습니다.
아래 항목을 실제 기기에서 테스트 후 결과를 공유 부탁드립니다:
- 키보드 열림 시 하단 버튼(IMPRESSION 입력) 가림 여부
- 제스처 내비게이션 vs 3-버튼 내비게이션 환경 모두에서 하단 안전영역 반영 여부
- 상태바/노치 영역과 상단 AppBar 겹침 여부
(참고: ReedFullScreen 정의 위치 → core/ui/src/main/kotlin/com/ninecraft/booket/core/ui/component/ReedFullScreen.kt:35)
ReedScaffold내 modifier.keyboardHide() 때문에 발생하는 문제이므로, 기본 Scaffold 사용하면 됨
easyhooon
approved these changes
Aug 9, 2025
Contributor
easyhooon
left a comment
easyhooon
left a comment
There was a problem hiding this comment.
ReedFullScreen이 Column기반이라 Column으로 한번더 감싸지 않아도 되기도하고,
기존 ReedScaffold의 modifier.keyboardHide() 때문에 발생하는 문제라, 기본 Scaffold 사용하는 방식이 나을것같아 수정했습니다
Contributor
Author
좋습니다~! 머지할게요 👍 |
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
2 participants
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.
🔗 관련 이슈
📙 작업 설명
💬 추가 설명 or 리뷰 포인트
Summary by CodeRabbit