Skip to content

[FEAT] 텍스트 선택 핸들 및 하이라이트 색상 통일 - #980

Closed
Sadturtleman wants to merge 1 commit into
developfrom
feat/text-selection-color
Closed

Sadturtleman wants to merge 1 commit into
developfrom
feat/text-selection-color

Conversation

@Sadturtleman

@Sadturtleman Sadturtleman commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

📌𝘐𝘴𝘴𝘶𝘦𝘴

  • 컬렉션 작성 개발 추가 요청 (Android: EditText 텍스트 선택 색상 통일)

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

컬렉션 작성 화면에서 텍스트를 선택할 때 나오는 핸들과 하이라이트 색을 피드 작성 화면과 같게 맞췄습니다.

원인

  • 컬렉션 입력창(Compose BasicTextField)은 커서 색만 Primary100으로 지정하고, 핸들과 하이라이트는 MaterialTheme 기본값(colorScheme.primary 기준)을 쓰고 있었습니다.
  • WebsosoTheme은 dynamicColor = true가 기본값이라, Android 12 이상에서는 핸들과 하이라이트가 배경화면 색을 따라갔습니다.
  • Android 11 이하에서도 하이라이트가 Primary100에 알파 40%를 준 색이라, 피드의 Primary50과 달랐습니다.

변경

  • WebsosoTheme에서 LocalTextSelectionColors를 handleColor = Primary100, backgroundColor = Primary50으로 지정했습니다. 피드 작성 화면의 colorControlActivated, textColorHighlight와 같은 색입니다.
  • 테마 전역에 적용했기 때문에 컬렉션 이름, 설명, 작품 검색 입력과 함께 탐색 > 키워드 검색창도 같은 색으로 바뀝니다.
  • 커서는 기존대로 Primary100(보라)을 유지합니다.

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

  • :core:designsystem, :feature:collection 컴파일과 ktlintCheck는 통과했고, 실기기 확인은 아직입니다.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 개선 사항
    • 텍스트 선택 시 선택 핸들 색상이 기본 테마의 주요 색상으로 표시됩니다.
    • 텍스트 선택 영역의 배경이 테마에 맞는 색상으로 표시됩니다.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 0a96b307-dfde-4552-9d5d-be142111fa26
📥 Commits

Reviewing files that changed from the base of the PR and between f9920de and 061fee0.

📒 Files selected for processing: 1
  • core/designsystem/src/main/java/com/into/websoso/core/designsystem/theme/Theme.kt

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

WebsosoTheme이 MaterialTheme 내부에서 텍스트 선택 색상을 제공합니다. 선택 핸들에는 Primary100을, 선택 배경에는 Primary50을 설정합니다.

Changes

테마 텍스트 선택 색상

Layer / File(s) Summary
텍스트 선택 색상 제공
core/designsystem/src/main/java/com/into/websoso/core/designsystem/theme/Theme.kt
Compose 텍스트 선택 API를 추가하고, CompositionLocalProvider에서 선택 핸들과 배경 색상을 각각 Primary100, Primary50으로 지정합니다. 콘텐츠는 색상 제공자 내부에서 렌더링됩니다.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Feature

Merge Risk: ⚪ Minimal · up to 061fe

The change is mergeable; only physical-device validation remains as normal follow-up.

Pre-merge checks: Passed 4 · Failed 1

Failed 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%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
Passed Passed checks: 4 passed
Check name Status Explanation
Title check Passed PR 제목은 텍스트 선택 핸들과 하이라이트 색상을 통일하는 주요 변경 사항을 정확하고 간결하게 설명합니다.
Description check Passed PR 설명은 이슈, 작업 내용, 원인, 변경 사항, 스크린샷, 리뷰어 안내를 포함합니다. 실기기 검증이 아직 완료되지 않았고 이슈 번호가 없지만, 핵심 변경 내용과 검증 상태는 충분히 설명되어 있습니다.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches: 💡 1
📝 Generate docstrings: 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

토끼가 색을 골라 담네
핸들은 산뜻한 Primary100
배경은 포근한 Primary50
테마 안에 나란히 놓고
글자를 살며시 선택하네

Comment @coderabbitai help to get the list of available commands.

@Sadturtleman
Sadturtleman deleted the feat/text-selection-color branch October 21, 2026 15:38
@Sadturtleman

Copy link
Copy Markdown
Contributor Author

브랜치명을 이슈 컨벤션(feat/981)으로 변경하면서 이 PR이 닫혀 #982 로 다시 올렸습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant