Skip to content

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

Merged
Sadturtleman merged 1 commit into
developfrom
feat/981
Oct 24, 2026
Merged

Sadturtleman merged 1 commit into
developfrom
feat/981

Conversation

@Sadturtleman

@Sadturtleman Sadturtleman commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

📌𝘐𝘴𝘴𝘶𝘦𝘴

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

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

원인

  • 컬렉션 입력창(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(보라)을 유지합니다.

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

변경 전 변경 후
before after

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

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 스타일
    • 텍스트 선택 시 선택 영역과 선택 핸들의 색상이 테마에 맞게 표시됩니다.
    • 선택 배경에는 Primary50, 선택 핸들에는 Primary100 색상이 적용됩니다.

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: e6cec508-4407-45cc-813e-bf5225226411
📥 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에서 텍스트 선택 색상을 설정합니다. 선택 핸들은 Primary100, 선택 배경은 Primary50을 사용합니다.

Changes

텍스트 선택 테마

Layer / File(s) Summary
텍스트 선택 색상 제공
core/designsystem/src/main/java/com/into/websoso/core/designsystem/theme/Theme.kt
WebsosoTheme의 MaterialTheme 콘텐츠를 LocalTextSelectionColors로 감쌉니다. 핸들 색상은 Primary100, 선택 배경 색상은 Primary50으로 설정합니다.

Priority: ⬇️ Low

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

Change: Bug fix · Severity of issue fixed: Low

Merge Risk: ⚪ Minimal · up to 061fe

No actionable merge-blocking risk remains.

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 제목은 텍스트 선택 핸들과 하이라이트 색상을 통일하는 주요 변경 사항을 정확하고 간결하게 설명합니다.
Linked Issues check Passed #981의 세 가지 코딩 요구사항을 모두 충족합니다. WebsosoTheme이 LocalTextSelectionColors를 전역으로 제공합니다. 핸들 색은 Primary100이고 선택 배경 색은 Primary50입니다. 이 값은 dynamicColor로 계산한 MaterialTheme 색상과 독립적이므로 Android 12 이상에서 …
Out of Scope Changes check Passed 변경은 Theme.kt의 텍스트 선택 색상 설정으로 한정됩니다. 전역 WebsosoTheme 적용은 #981이 명시한 Compose 입력창 범위와 일치합니다. 관련 없는 변경은 확인되지 않았습니다.
Description check Passed PR 설명은 템플릿의 필수 섹션을 모두 포함합니다. 이슈 번호, 작업 내용과 원인, 변경 사항, 스크린샷을 구체적으로 작성했습니다. 리뷰어 안내 섹션은 비어 있지만 비필수 내용이므로 설명은 충분합니다.
Full details: Linked Issues check

Explanation

#981의 세 가지 코딩 요구사항을 모두 충족합니다. WebsosoTheme이 LocalTextSelectionColors를 전역으로 제공합니다. 핸들 색은 Primary100이고 선택 배경 색은 Primary50입니다. 이 값은 dynamicColor로 계산한 MaterialTheme 색상과 독립적이므로 Android 12 이상에서 배경화면 색을 사용하지 않습니다. 커서 설정은 변경하지 않았습니다.

✨ 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 merged commit f07e6d5 into develop Oct 24, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🍯 [FEAT] 새로운 기능을 개발합니다. 🧙마법사 하루

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: 텍스트 선택 핸들 및 하이라이트 색상 통일

2 participants