Skip to content

[Refactor/#127] BasicTextField 수정#177

Merged
nahy-512 merged 6 commits intodevelopfrom
refactor/#127-basictextfield-edit
Jan 22, 2026
Merged

[Refactor/#127] BasicTextField 수정#177
nahy-512 merged 6 commits intodevelopfrom
refactor/#127-basictextfield-edit

Conversation

@nahy-512
Copy link
Contributor

@nahy-512 nahy-512 commented Jan 22, 2026

📮 관련 이슈

📌 작업 내용

  • 글자수에서 이모지 제외
  • singleLine 속성 추가

📸 스크린샷

스크린샷

나중에 추가할게요..

😅 미구현

  • decorationBox 추가 (글자수에 따라 텍필 늘어나게)

🫛 To. 리뷰어

Summary by CodeRabbit

  • 새로운 기능

    • 텍스트 입력 필드에 키보드 액션(검색/다음/완료) 및 onAction 콜백 지원 추가
    • 입력 필드에 single-line 옵션 추가
  • 개선사항

    • 사용자 인지 기반의 문자열 길이(grapheme) 계산 적용으로 다국어(예: 한글) 입력 길이 표시 정확도 향상
    • 키보드 네비게이션 및 입력 편의성 강화

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

@nahy-512 nahy-512 self-assigned this Jan 22, 2026
@nahy-512 nahy-512 requested a review from a team as a code owner January 22, 2026 16:09
@nahy-512 nahy-512 added the Fix 🐛 버그, UI 오류, 오타, 명세와의 불일치 등을 수정 label Jan 22, 2026
@nahy-512 nahy-512 linked an issue Jan 22, 2026 that may be closed by this pull request
2 tasks
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 22, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

텍스트 필드의 길이 검증을 문자(코드 유닛)에서 그래핌 길이로 전환하고, 텍스트 컴포저블들에 singleLine·keyboardOptions·keyboardActions 파라미터를 추가했으며, Pebble 라이브러리 종속성을 추가했습니다. 여러 화면의 입력 필드들이 키보드 IME 구성으로 업데이트되었습니다.

Changes

코호트 / 파일(s) 변경 요약
의존성 및 버전 관리
app/build.gradle.kts, gradle/libs.versions.toml
Pebble 라이브러리(io.github.chattymin:pebble:0.1.0) 의존성 추가
기본 텍스트필드 컴포넌트
app/src/main/java/com/flint/core/designsystem/component/textfield/FlintBasicTextField.kt
입력 길이 체크를 graphemeLength로 변경, 공개 파라미터 singleLine: Boolean = true 추가 및 BasicTextField에 전달
긴 텍스트필드 컴포넌트
app/src/main/java/com/flint/core/designsystem/component/textfield/FlintLongTextField.kt
singleLine, keyboardOptions, keyboardActions 공개 파라미터 추가, 길이 표시를 graphemeLength 기반으로 변경
컬렉션 생성 화면 관련
app/src/main/java/com/flint/presentation/collectioncreate/AddContentScreen.kt, app/src/main/java/com/flint/presentation/collectioncreate/CollectionCreateScreen.kt, app/src/main/java/com/flint/presentation/collectioncreate/component/CollectionCreateContentItemList.kt
검색/타이틀/설명 입력에 IME 액션·keyboardOptions/keyboardActions 전달 추가, 일부 필드에 singleLine 설정(단일/여러 줄) 적용
온보딩 화면 및 프리뷰
app/src/main/java/com/flint/presentation/onboarding/OnboardingContentScreen.kt, app/src/main/java/com/flint/presentation/onboarding/OnboardingProfileScreen.kt
프리뷰 상태 바인딩을 stateful로 변경(remember), 닉네임 길이 표시를 graphemeLength로 변경 및 singleLine 적용

Sequence Diagram(s)

(생성 조건 미충족 — 생략)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested reviewers

  • kimjw2003

Poem

🐰 그래핌으로 세어본 글자들,
키보드는 살짝 고개를 끄덕여,
한 줄도, 여러 줄도 즐겁게,
Pebble은 조용히 반짝이고,
화면 위 입력은 더 부드럽게 춤춘다. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ 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%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed 제목은 주요 변경사항인 BasicTextField 수정을 반영하고 있으며, 리팩토링과 이슈 번호를 포함하여 충분히 명확합니다.
Description check ✅ Passed 설명서는 템플릿 구조를 따르고 있으나, 관련 이슈 번호가 플레이스홀더로 남아있고 스크린샷이 제공되지 않았습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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 and usage tips.

Copy link
Contributor

@kimjw2003 kimjw2003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nahy-512 nahy-512 merged commit d8f1d6e into develop Jan 22, 2026
1 of 2 checks passed
@nahy-512 nahy-512 deleted the refactor/#127-basictextfield-edit branch January 22, 2026 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix 🐛 버그, UI 오류, 오타, 명세와의 불일치 등을 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] BasicTextField 수정

2 participants

Comments