You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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으로 설정합니다.
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 checks: 4 passed
Check name
Status
Explanation
Title check
제목은 텍스트 선택 핸들과 하이라이트 색상을 통일하는 주요 변경 사항을 정확하고 간결하게 설명합니다.
Linked Issues check
#981의 세 가지 코딩 요구사항을 모두 충족합니다. WebsosoTheme이 LocalTextSelectionColors를 전역으로 제공합니다. 핸들 색은 Primary100이고 선택 배경 색은 Primary50입니다. 이 값은 dynamicColor로 계산한 MaterialTheme 색상과 독립적이므로 Android 12 이상에서 …
Out of Scope Changes check
변경은 Theme.kt의 텍스트 선택 색상 설정으로 한정됩니다. 전역 WebsosoTheme 적용은 #981이 명시한 Compose 입력창 범위와 일치합니다. 관련 없는 변경은 확인되지 않았습니다.
Description check
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.
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
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.
📌𝘐𝘴𝘴𝘶𝘦𝘴
https://discord.com/channels/1185753783020568648/1527684314593234995
📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯
컬렉션 작성 화면에서 텍스트를 선택할 때 나오는 핸들과 하이라이트 색을 피드 작성 화면과 같게 맞췄습니다.
원인
BasicTextField)은 커서 색만Primary100으로 지정하고, 핸들과 하이라이트는MaterialTheme기본값(colorScheme.primary기준)을 쓰고 있었습니다.WebsosoTheme은dynamicColor = true가 기본값이라, Android 12 이상에서는 핸들과 하이라이트가 배경화면 색을 따라갔습니다.Primary100에 알파 40%를 준 색이라, 피드의Primary50과 달랐습니다.변경
WebsosoTheme에서LocalTextSelectionColors를handleColor = Primary100,backgroundColor = Primary50으로 지정했습니다. 피드 작성 화면의colorControlActivated,textColorHighlight와 같은 색입니다.Primary100(보라)을 유지합니다.📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵
💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴
🤖 Generated with Claude Code
Summary by CodeRabbit