PrezelChip 변경사항 반영, 아이콘, ColorToken 변경#97
Merged
HamBeomJoon merged 10 commits intodevelopfrom Apr 8, 2026
Merged
Conversation
* `ic_script` 아이콘 애셋 추가 (`core_designsystem_ic_script.xml`)
PrezelColorScheme 내의 배경색 관련 토큰 값을 디자인 가이드에 맞춰 조정했습니다. * `bgLarge` 색상을 `CoolGray50`에서 `CoolGray100`으로 변경 * `bgDisabled` 색상을 `CoolGray100`에서 `CoolGray50`으로 변경
* refactor: PrezelChipStyle의 상태별 색상 매핑 수정
디자인 시스템 가이드에 맞춰 `PrezelChip`의 배경색과 아이콘 색상 결정 로직을 개선했습니다.
* **배경색(`containerColor`)**:
* `DISABLED` 상태의 배경색을 `bgLarge`에서 `bgDisabled`로 변경했습니다.
* `FILLED` 타입의 기본 배경색을 `bgMedium`에서 `bgLarge`로 변경했습니다.
* **아이콘 색상(`iconColor`)**:
* `FILLED` 타입일 때 기본 아이콘 색상을 `iconMedium`으로, `OUTLINED` 타입일 때 `iconRegular`를 사용하도록 세분화했습니다.
* `build.gradle.kts` 내 `Detekt` 태스크 설정에 `jvmTarget = "21"`을 명시적으로 추가했습니다. * `io.gitlab.arturbosch.detekt.Detekt` 임포트문을 추가하고 관련 코드를 정리했습니다.
* refactor: PrezelChip 아이콘 및 텍스트 색상 관리 로직 분리 `PrezelChip`의 내부 색상을 결정하는 `PrezelChipColors` 구조를 개선하여 아이콘과 텍스트 색상을 개별적으로 설정할 수 있도록 변경했습니다. * `PrezelChipColors`에서 `contentColor`를 제거하고 `iconColor`와 `textColor`로 세분화 * `LocalPrezelChipTextColor`, `LocalPrezelChipIconColor` 등 전용 `CompositionLocal` 추가 * `PrezelChipStyle` 내에서 상호작용 및 피드백 상태에 따른 `iconColor()`와 `textColor()` 계산 로직 분리 구현 * `PrezelChip` 컴포저블에서 `Icon`의 `tint`와 `Text`의 `color`에 각각 분리된 색상을 적용하도록 수정 * 기존 프리뷰 코드의 커스텀 컬러 설정을 변경된 구조에 맞게 업데이트 (예: `feedbackWarningRegular`를 아이콘과 텍스트 모두에 적용)
Contributor
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthrough칩 컴포넌트 API와 스타일 구현을 재구성(스타일 파일 제거 → Defaults/Models/Layout로 분리), 색상 토큰 교체, 새 스크립트 아이콘 추가, 그리고 Gradle Detekt 작업에 Changes
Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
🧹 Nitpick comments (1)
Prezel/core/designsystem/src/main/res/drawable/core_designsystem_ic_script.xml (1)
7-7: 아이콘 색상 하드코딩으로 상태별 tint 적용이 제한될 수 있습니다.
android:fillColor="#6E737D"고정값은 컴포넌트에서 분리한iconColor정책과 어긋날 수 있습니다. 아이콘은 기본적으로 중립 색(예: black) + 런타임 tint 적용 또는 디자인시스템 color 리소스 참조로 맞추는 편이 안전합니다.예시 수정안
- android:fillColor="#6E737D" + android:fillColor="@android:color/black"또는
- android:fillColor="#6E737D" + android:fillColor="@color/core_designsystem_icon_medium"🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Prezel/core/designsystem/src/main/res/drawable/core_designsystem_ic_script.xml` at line 7, Replace the hardcoded color in the vector drawable by removing or replacing the android:fillColor="#6E737D" entry in core_designsystem_ic_script.xml so the icon can be tinted at runtime; either set the fillColor to a neutral default (e.g., black or `@android`:color/black) or reference a design-system color resource/attribute (e.g., `@color/icon_neutral` or ?attr/colorControlNormal) so components can apply state-based tints. Ensure the drawable remains a single-color vector (no multiple hardcoded fills) so ImageView/AppCompatImageView tinting works as expected.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In
`@Prezel/core/designsystem/src/main/res/drawable/core_designsystem_ic_script.xml`:
- Line 7: Replace the hardcoded color in the vector drawable by removing or
replacing the android:fillColor="#6E737D" entry in
core_designsystem_ic_script.xml so the icon can be tinted at runtime; either set
the fillColor to a neutral default (e.g., black or `@android`:color/black) or
reference a design-system color resource/attribute (e.g., `@color/icon_neutral` or
?attr/colorControlNormal) so components can apply state-based tints. Ensure the
drawable remains a single-color vector (no multiple hardcoded fills) so
ImageView/AppCompatImageView tinting works as expected.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: c077f25f-5297-4154-8203-14d151e600e9
📒 Files selected for processing (5)
Prezel/build.gradle.ktsPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChip.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipStyle.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/theme/PrezelColorScheme.ktPrezel/core/designsystem/src/main/res/drawable/core_designsystem_ic_script.xml
* refactor: PrezelChip 스타일 계산 로직 분리 및 Defaults 도입 기존 `PrezelChipStyle` 클래스에 집중되어 있던 스타일 계산 로직을 `PrezelChipDefaults` 객체로 분리하여 컴포넌트 구조를 개선했습니다. * `PrezelChipStyle.kt` 삭제 및 관련 로직을 `PrezelChipDefaults.kt`로 이동 * `PrezelChipDefault` 데이터 클래스를 통해 계산된 스타일 값(`shape`, `border`, `color` 등)을 한 번에 전달하도록 변경 * `PrezelChipDefaults.getDefault()`를 통해 상태별(Type, Size, Interaction, Feedback) 스타일 값을 동적으로 계산 * feat: PrezelChip 모델 정의 및 주석 추가 칩의 상태와 속성을 정의하는 enum 클래스들을 별도 파일로 분리하고 각 항목에 대한 설명을 추가했습니다. * `PrezelChipModels.kt` 신규 생성 * `PrezelChipType`, `PrezelChipSize`, `PrezelChipInteraction`, `PrezelChipFeedback` 이동 및 KDoc 주석 추가 * refactor: PrezelChip 및 PrezelIconChip 파라미터 구조 변경 `style: PrezelChipStyle` 객체를 통째로 받던 방식에서 개별 상태 값(`type`, `size` 등)을 직접 받도록 변경하여 사용성을 높였습니다. * `PrezelChip`, `PrezelIconChip`의 파라미터를 `type`, `size`, `interaction`, `feedback`으로 명시화 * `PrezelChip` 내부에서 `PrezelChipDefaults.getDefault`를 호출하여 스타일 설정 적용 * `PrezelChipIcon` 컴포저블이 `PrezelChipDefault` 설정을 주입받아 렌더링하도록 수정 * refactor: PrezelChip 프리뷰 코드 구조 개선 및 분리 반복되는 프리뷰 로직을 별도의 파일로 추출하여 가독성을 높이고 유지보수를 용이하게 했습니다. * `PrezelChipPreview.kt` 생성 및 공통 프리뷰 콘텐츠 함수(`PrezelChipPreviewContent` 등) 정의 * `PrezelChip.kt` 및 `PrezelIconChip.kt` 내 프리뷰 코드를 신규 분리된 함수를 사용하도록 리팩터링 * 사용자 정의 색상 테스트를 위한 `PrezelChipCustomPreviewContent` 추가
Contributor
There was a problem hiding this comment.
🧹 Nitpick comments (3)
Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipDefaults.kt (1)
112-119: Border 색상에chipColors.iconColor를 사용하는 로직이 의도된 것인지 확인 필요합니다.
OUTLINED타입의 border 색상 결정 시chipColors.iconColor를 참조하고 있습니다. 일관성을 위해 별도의borderColor필드를PrezelChipColors에 추가하거나, 이 동작이 의도된 것이라면 주석으로 명시하는 것이 좋겠습니다.val borderColor = when { chipColors.iconColor != Color.Unspecified -> chipColors.iconColor // iconColor를 border에도 사용 ... }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipDefaults.kt` around lines 112 - 119, The border color logic in PrezelChipDefaults (the borderColor when computing OUTLINED chips) currently falls back to chipColors.iconColor (chipColors.iconColor) which may be unintended; either add a dedicated border color property to PrezelChipColors (e.g., borderColor) and use that in the borderColor when present, or if using iconColor for borders is intentional, add a clarifying comment in PrezelChipDefaults next to the chipColors.iconColor branch to document this behavior and update any constructors/builders of PrezelChipColors to include/handle the new borderColor field if you add it.Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipPreview.kt (1)
109-112: 하드코딩된 색상 값을 테마 토큰으로 대체하는 것을 고려해 주세요.프리뷰 코드이므로 큰 문제는 아니지만,
Color(0xFFDBFFF6)와Color(0xFF00A37A)같은 하드코딩 값이 디자인 시스템의 토큰과 일치하는지 확인하고, 가능하다면 테마 토큰으로 대체하는 것이 일관성 유지에 도움됩니다.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipPreview.kt` around lines 109 - 112, The preview is using hardcoded Color literals in PrezelChipPreview when constructing PrezelChipColors (containerColor = Color(0xFFDBFFF6), textColor = Color(0xFF00A37A)); replace these literals with the appropriate design system theme tokens (e.g., use the color tokens exposed by your theme/provider such as PrezelTheme.colors.<tokenName> or DesignTokens.<tokenName>) so PrezelChipPreview and the PrezelChipColors instance reflect the centralized theme values and maintain consistency.Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelIconChip.kt (1)
13-33:config파라미터와 개별 파라미터 간 불일치 가능성이 있습니다.호출자가
config를 직접 제공하면서 동시에type,size등의 개별 파라미터도 다른 값으로 전달할 경우,config의 스타일과PrezelChip에 전달되는type/size/interaction/feedback값이 불일치할 수 있습니다.♻️ 권장 수정안: config 제공 시 개별 파라미터 무시 또는 문서화
`@Composable` fun PrezelIconChip( `@DrawableRes` iconResId: Int, modifier: Modifier = Modifier, type: PrezelChipType = PrezelChipType.FILLED, size: PrezelChipSize = PrezelChipSize.REGULAR, interaction: PrezelChipInteraction = PrezelChipInteraction.DEFAULT, feedback: PrezelChipFeedback = PrezelChipFeedback.DEFAULT, - config: PrezelChipDefault = PrezelChipDefaults.getDefault( + config: PrezelChipDefault? = null, +) { + val resolvedConfig = config ?: PrezelChipDefaults.getDefault( iconOnly = true, type = type, size = size, interaction = interaction, feedback = feedback, - ), -) { + ) + PrezelChip( modifier = modifier, iconResId = iconResId, type = type, size = size, interaction = interaction, feedback = feedback, - config = config, + config = resolvedConfig, ) }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelIconChip.kt` around lines 13 - 33, PrezelIconChip may pass mismatched styling because callers can supply a custom config while also passing different type/size/interaction/feedback; update PrezelIconChip so it enforces consistency by deriving type, size, interaction and feedback from the provided config (config from PrezelChipDefaults.getDefault or a caller-supplied config) and pass those derived values into PrezelChip instead of the independent params, or alternatively make config nullable (default null) and if non-null ignore the individual params (or validate and throw/log on mismatch). Locate PrezelIconChip and the call to PrezelChip as well as PrezelChipDefaults.getDefault to implement the chosen approach.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipDefaults.kt`:
- Around line 112-119: The border color logic in PrezelChipDefaults (the
borderColor when computing OUTLINED chips) currently falls back to
chipColors.iconColor (chipColors.iconColor) which may be unintended; either add
a dedicated border color property to PrezelChipColors (e.g., borderColor) and
use that in the borderColor when present, or if using iconColor for borders is
intentional, add a clarifying comment in PrezelChipDefaults next to the
chipColors.iconColor branch to document this behavior and update any
constructors/builders of PrezelChipColors to include/handle the new borderColor
field if you add it.
In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipPreview.kt`:
- Around line 109-112: The preview is using hardcoded Color literals in
PrezelChipPreview when constructing PrezelChipColors (containerColor =
Color(0xFFDBFFF6), textColor = Color(0xFF00A37A)); replace these literals with
the appropriate design system theme tokens (e.g., use the color tokens exposed
by your theme/provider such as PrezelTheme.colors.<tokenName> or
DesignTokens.<tokenName>) so PrezelChipPreview and the PrezelChipColors instance
reflect the centralized theme values and maintain consistency.
In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelIconChip.kt`:
- Around line 13-33: PrezelIconChip may pass mismatched styling because callers
can supply a custom config while also passing different
type/size/interaction/feedback; update PrezelIconChip so it enforces consistency
by deriving type, size, interaction and feedback from the provided config
(config from PrezelChipDefaults.getDefault or a caller-supplied config) and pass
those derived values into PrezelChip instead of the independent params, or
alternatively make config nullable (default null) and if non-null ignore the
individual params (or validate and throw/log on mismatch). Locate PrezelIconChip
and the call to PrezelChip as well as PrezelChipDefaults.getDefault to implement
the chosen approach.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: f35a628b-d681-4d85-b853-e275f2b1d6a3
📒 Files selected for processing (6)
Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChip.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipDefaults.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipModels.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipPreview.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipStyle.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelIconChip.kt
💤 Files with no reviewable changes (1)
- Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipStyle.kt
✅ Files skipped from review due to trivial changes (1)
- Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipModels.kt
* refactor: PrezelIconChip의 config 파라미터 초기화 로직 수정 `PrezelIconChip` 컴포저블의 `config` 파라미터 기본값을 `null`로 변경하고, 내부에서 `resolvedConfig`를 통해 초기화하도록 수정했습니다. 이는 파라미터로 전달된 값이 없을 때만 `PrezelChipDefaults.getDefault`가 호출되도록 보장합니다. * feat: PrezelChipColors 내 borderColor 필드 추가 및 적용 `PrezelChipColors` 데이터 클래스에 `borderColor` 필드를 추가하여 테두리 색상을 개별적으로 지정할 수 있도록 개선했습니다. * `PrezelChipColors`: `borderColor` 프로퍼티 추가 * `PrezelChipDefaults`: `borderColor`가 명시적으로 지정된 경우 이를 우선적으로 사용하도록 로직 수정 (기존에는 `iconColor`를 참조함) * `PrezelChipPreview`: 커스텀 컬러 테스트 케이스에 `borderColor` 설정 추가
HamBeomJoon
added
🔨 refactor
`PrezelChip`의 가독성과 유지보수성을 높이기 위해 내부 상태 관리 방식을 변경하고 관련 파일을 정리하였습니다.
* **refactor: Chip 설정 및 모델 구조 개선**
* `PrezelChipModels.kt`와 `PrezelChipDefaults.kt`를 `config` 패키지로 이동하여 구조화했습니다.
* `CompositionLocal`(`LocalPrezelChipColors` 등)을 통한 상태 전달 방식을 제거하고, `config` 객체를 직접 전달하거나 확장 함수를 통해 속성을 적용하도록 변경했습니다.
* `PrezelChipDefault.withCustomColors` 확장 함수를 추가하여 사용자 정의 색상 적용 로직을 캡슐화했습니다.
* **refactor: PrezelChip 및 PrezelIconChip 구현 최적화**
* `PrezelChip`의 중복 정의된 함수를 통합하고, 실제 UI 렌더링을 담당하는 `PrezelChipContent`를 분리했습니다.
* `PrezelIconChip`에서 `config` 파라미터의 기본값 설정 로직을 개선했습니다.
* 불필요한 `LocalTextStyle` 및 `LocalPrezelChipIconColor` 등의 사용을 제거하고 `config`에서 직접 값을 참조하도록 수정했습니다.
* **style: 미리보기(Preview) 코드 통합 및 개선**
* `PrezelChipPreview.kt` 파일을 삭제하고, 해당 로직을 각 컴포넌트 파일(`PrezelChip.kt`, `PrezelIconChip.kt`) 내부로 통합했습니다.
* `PreviewParameterProvider`를 도입하여 다양한 `PrezelChipType`에 대한 미리보기를 효율적으로 구성했습니다.
* 미리보기 레이아웃을 `PreviewSection`과 `PreviewValueRow`를 사용하는 일관된 스타일로 업데이트했습니다.
Contributor
There was a problem hiding this comment.
🧹 Nitpick comments (1)
Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/config/PrezelChipDefaults.kt (1)
203-214: when 표현식 순서 일관성 (선택 사항)
getIconTextSpacing은REGULAR→SMALL순서이고,getIconSize는SMALL→REGULAR순서입니다. 기능적으로는 문제없지만, 가독성을 위해 순서를 통일하면 좋습니다.♻️ 순서 통일 제안
`@Composable` private fun getIconTextSpacing(size: PrezelChipSize): Dp = when (size) { - PrezelChipSize.REGULAR -> PrezelTheme.spacing.V4 PrezelChipSize.SMALL -> PrezelTheme.spacing.V2 + PrezelChipSize.REGULAR -> PrezelTheme.spacing.V4 }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/config/PrezelChipDefaults.kt` around lines 203 - 214, The two when-expressions have inconsistent case ordering: getIconTextSpacing lists PrezelChipSize.REGULAR then SMALL, while getIconSize lists SMALL then REGULAR; update getIconSize (or getIconTextSpacing) so both when clauses use the same case order (preferably REGULAR then SMALL) to improve readability—modify the when in getIconSize/getIconTextSpacing accordingly while keeping the same return values.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/config/PrezelChipDefaults.kt`:
- Around line 203-214: The two when-expressions have inconsistent case ordering:
getIconTextSpacing lists PrezelChipSize.REGULAR then SMALL, while getIconSize
lists SMALL then REGULAR; update getIconSize (or getIconTextSpacing) so both
when clauses use the same case order (preferably REGULAR then SMALL) to improve
readability—modify the when in getIconSize/getIconTextSpacing accordingly while
keeping the same return values.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 6b3ee211-222a-406a-97d0-b3f541e9be8d
📒 Files selected for processing (4)
Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChip.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelIconChip.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/config/PrezelChipDefaults.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/config/PrezelChipModels.kt
🚧 Files skipped from review as they are similar to previous changes (2)
- Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelIconChip.kt
- Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChip.kt
`PrezelChip`과 `PrezelIconChip`의 공통 UI 렌더링 로직을 `PrezelChipLayout`으로 통합하고, 커스텀 스타일 적용 방식을 개선했습니다.
moondev03
approved these changes
Apr 8, 2026
Member
Chip
IconChip
|
* refactor: PrezelChipType에 따른 borderColor 조건부 적용 `PrezelChipDefaults`에서 `borderColor`를 설정할 때, 칩의 타입이 `PrezelChipType.OUTLINED`인 경우에만 색상을 할당하고 그 외의 경우에는 `null`이 전달되도록 로직을 수정하였습니다.
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/config/PrezelChipLayout.kt`:
- Around line 56-60: 현재 Text( text = text, color = config.textColor, style =
config.textStyle )가 긴 문구로 인해 칩 높이를 늘리거나 레이아웃을 깨뜨릴 수 있으니 PrezelChipLayout의 해당
Text 컴포저블에 maxLines = 1 및 overflow = TextOverflow.Ellipsis 속성을 추가해 한 줄로 고정하고 넘치는
텍스트는 말줄임표로 처리되도록 수정하세요; TextOverflow를 사용하려면 필요한 import도 추가해 주세요.
- Around line 67-78: PrezelChipIcon always passes contentDescription = null
which makes icon-only chips nameless for TalkBack; update the API to accept a
contentDescription parameter and thread it from PrezelIconChip into
PrezelChipIcon (add a contentDescription: String? parameter to PrezelChipIcon
and to the PrezelIconChip/PrezelChipLayout call sites), ensure when a chip has
visible text you keep contentDescription = null (decorative), and when using the
icon-only path require/forward a non-null description so TalkBack receives an
accessible label.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 3259af62-f271-4715-8c3c-482dd23f8d72
📒 Files selected for processing (4)
Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChip.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelIconChip.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/config/PrezelChipDefaults.ktPrezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/config/PrezelChipLayout.kt
...em/src/main/java/com/team/prezel/core/designsystem/component/chip/config/PrezelChipLayout.kt
Show resolved
Hide resolved
...em/src/main/java/com/team/prezel/core/designsystem/component/chip/config/PrezelChipLayout.kt
Show resolved
Hide resolved
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.
📌 작업 내용
icon/script 추가
bg_large: cool gray_50 -> cool gray_100 변경
bg_disabled: cool gray_100 -> cool gray_50 변경
type = filled, size = regular, interaction = default, feedback = default인 PrezelChip
Chip의 contentColor를 하나로받고 있었는데, iconColor, textColor로 분리했습니다.
🧩 관련 이슈
📸 스크린샷
📢 논의하고 싶은 내용
Summary by CodeRabbit
릴리스 노트
새로운 기능
버그 수정
Chores