PrezelButton 구현

[moondev03]

📌 작업 내용

Prezel 서비스 테마를 사용한 기본 컴포넌트 PrezelButton, PrezelTextButton, PrezelIconButton을 구현했습니다.

Prezel Button Property

enum class PrezelButtonType {
    FILLED,
    OUTLINED,
    GHOST,
}

enum class PrezelButtonHierarchy {
    PRIMARY,
    SECONDARY,
}

enum class PrezelButtonSize {
    XSMALL,
    SMALL,
    REGULAR,
}

@Immutable
data class PrezelButtonStyle(
    val buttonType: PrezelButtonType = PrezelButtonType.FILLED,
    val buttonHierarchy: PrezelButtonHierarchy = PrezelButtonHierarchy.PRIMARY,
    val buttonSize: PrezelButtonSize = PrezelButtonSize.REGULAR,
    val isRounded: Boolean = false,
)

각 Property에 맞는 디자인을 반영하기 위한 코드는 PrezelButtonStyle에 구현했습니다.
원하는 스타일의 버튼을 사용하기 위해서는 PrezelButtonStyle를 정의하여 사용하시면 됩니다.

IconSource 구현

Painter는 추상 클래스이기 때문에 컴파일 타임에 안정성(Stable)을 판단할 수 없습니다.
이 문제를 해결하기 위해 IconSource와 이를 구현한 DrawableIcon을 도입했습니다.

이 구조를 통해 아이콘에 필요한 Painter와 contentDescription 그리고 리소스 정보를 하나의 객체로 일관되게 관리할 수 있습니다.
그 결과, 아이콘을 사용하는 컴포저블에서는 IconSource만 전달받아 iconSource.painter()와 iconSource.contentDescription처럼 명확하고 안전한 방식으로 사용할 수 있게 되었습니다.

이는 API 사용성을 개선할 뿐만 아니라, 아이콘 관련 책임을 한 곳에 모아 컴포저블의 역할을 더욱 단순하게 prove 데에도 도움이 됩니다.

기존 PrezelTopAppBar Preview 코드 개선

Preview 코드에 배경 색상을 적용했습니다.

🧩 관련 이슈

• closed PrezelButton 구현 #22

📸 스크린샷

PrezelButton (Icon + Text)

PrezelTextButton

PrezelIconButton

Summary by CodeRabbit

릴리스 노트

• 새로운 기능

  • 채워진, 아웃라인, 고스트 변형의 버튼 컴포넌트 추가 (아이콘/텍스트 조합 지원 및 크기별 레이아웃)
  • 아이콘 전용 버튼 및 텍스트 전용 버튼 API 추가
  • 버튼 스타일(계층, 크기, 라운드) 선택 기능 및 미리보기 화면 추가

• 스타일

  • 라이트 테마 경계 색상 조정 및 테마 로컬 텍스트/콘텐츠 컬러 제공 개선

• 잡무(Chores)

  • 정적 분석 설정 재구성 및 규칙 정리

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

[coderabbitai]

Walkthrough

디자인 시스템에 버튼 컴포넌트군을 추가합니다. 새 공개 Composable PrezelButton(onClick: () -> Unit, modifier: Modifier = Modifier, text: String? = null, icon: IconSource? = null, enabled: Boolean = true, style: PrezelButtonStyle = PrezelButtonStyle())을 도입하고 런타임으로 텍스트 또는 아이콘 중 적어도 하나가 존재함을 검증합니다. 버튼 스타일(enums와 데이터 클래스 포함), 아이콘 추상화(IconSource, DrawableIcon), 텍스트/아이콘 전용 래퍼(PrezelTextButton, PrezelIconButton), 그리고 다양한 미리보기 컴포저블 및 PrezelTheme에 대한 CompositionLocal 제공 변경이 추가되었습니다.

Possibly related PRs

• Color System 설계 및 적용 #10: PrezelTheme 및 색상 시스템 변경 — 버튼의 색상·테마 결정 로직이 해당 테마 인프라를 사용합니다.
• Number System 설계 및 구현 #18: 디자인 토큰(간격/모서리/스트로크) 추가 — 버튼의 패딩·모양·테두리 계산에서 토큰을 직접 참조합니다.
• Typography System 설계 및 적용 #13: 타이포그래피/LocalTextStyle 관련 변경 — 버튼이 제공하는 LocalTextStyle/타입그래피 활용과 연관됩니다.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Out of Scope Changes check	⚠️ Warning	TopAppBar.kt의 프리뷰 수정과 detekt-config.yml의 리포맷팅은 주요 목표와 무관한 변경입니다.	버튼 컴포넌트 구현과 무관한 TopAppBar 프리뷰 및 detekt 설정 변경을 제거하거나 별도 PR로 분리하세요.
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 (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 주요 변경 사항인 PrezelButton 구현을 명확하게 설명하고 있습니다.
Linked Issues check	✅ Passed	PR은 이슈 #22의 모든 주요 요구사항(PrezelButton, PrezelTextButton, PrezelIconButton, 스타일 변형)을 구현했습니다.
Description check	✅ Passed	PR 설명이 필요한 섹션을 모두 충실하게 작성했으며, 작업 내용, 관련 이슈, 스크린샷이 포함되어 있습니다.

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

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 3

🤖 Fix all issues with AI agents

In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/button/PrezelButton.kt`:
- Around line 27-68: PrezelButton에 iconContentDescription: String? 파라미터를 추가하고
내부에서 PrezelButtonIcon에 전달하도록 변경하라; 텍스트가 없는(hasText == false) 경우
iconContentDescription이 null이면 require를 사용해 예외를 발생시켜 스크린리더용 설명을 강제하고(예:
require(hasText || iconContentDescription != null) { ... }), PrezelButtonIcon 호출
시 contentDescription 파라미터로 전달하라; 동일한 iconContentDescription 파라미터를
PrezelIconButton에도 노출시키고 해당 호출부에서 전달되도록 동기화해 접근성 요구사항을 충족시켜라.

In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/button/PrezelButtonStyle.kt`:
- Around line 122-137: The function prezelButtonContentColor currently checks
isSystemInDarkTheme() and returns PrezelColorScheme.Dark.textLarge for
PrezelButtonType.FILLED, which ignores the injected theme and causes contrast
issues; change the logic to rely only on the theme-provided colors/flags (e.g.,
use the passed colors: PrezelColors and/or a theme flag such as
PrezelTheme.isDarkTheme or add a Boolean parameter isDarkTheme) instead of
isSystemInDarkTheme(), and replace any reference to
PrezelColorScheme.Dark.textLarge with the appropriate color from the injected
colors so FILLED text color is derived solely from the active PrezelTheme.

In `@Prezel/detekt-config.yml`:
- Around line 29-30: The rule entry "DestructuringDeclarationWithTooManyEntries"
has incorrect indentation; adjust it to match other rules by using two spaces
for the rule key level and two additional spaces for its properties so the block
reads with a 2-space indent for the rule name and its "active: false" property
indented two spaces further under that rule (i.e., follow the same 2-space rule
+ 2-space property nesting used by other rule entries).

🧹 Nitpick comments (2)

Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/button/PrezelIconButton.kt (1)

10-25: IconSource 추상화와 API 타입이 일관되지 않습니다.
추상화가 이미 도입된 만큼, 공개 API도 DrawableIcon 대신 IconSource를 받는 편이 확장성에 유리합니다.

♻️ 리팩터링 제안

-fun PrezelIconButton(
-    icon: DrawableIcon,
+fun PrezelIconButton(
+    icon: IconSource,
     onClick: () -> Unit,
     modifier: Modifier = Modifier,
     enabled: Boolean = true,
     style: PrezelButtonStyle = PrezelButtonStyle(),
 ) {

Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/button/PrezelButtonPreview.kt (1)

88-115: 불필요한 Column 래퍼 제거 고려.

Column 내부에 Row가 하나만 있어 불필요한 중첩으로 보입니다. 향후 확장 계획이 없다면 Row만 사용하는 것이 더 간결합니다.

♻️ 선택적 리팩토링 제안

`@Composable`
private fun PrezelButtonPreviewHierarchyBlock(
    type: PrezelButtonType,
    hierarchy: PrezelButtonHierarchy,
    enabled: Boolean,
    isRounded: Boolean,
    content: PrezelButtonPreviewContent,
    modifier: Modifier = Modifier,
) {
-    Column(
-        modifier = modifier,
-        verticalArrangement = Arrangement.spacedBy(8.dp),
-    ) {
-        Row(horizontalArrangement = Arrangement.spacedBy(8.dp)) {
-            PreviewSizes.forEach { size ->
-                content(
-                    PrezelButtonStyle(
-                        buttonType = type,
-                        buttonHierarchy = hierarchy,
-                        buttonSize = size,
-                        isRounded = isRounded,
-                    ),
-                    enabled,
-                )
-            }
+    Row(
+        modifier = modifier,
+        horizontalArrangement = Arrangement.spacedBy(8.dp),
+    ) {
+        PreviewSizes.forEach { size ->
+            content(
+                PrezelButtonStyle(
+                    buttonType = type,
+                    buttonHierarchy = hierarchy,
+                    buttonSize = size,
+                    isRounded = isRounded,
+                ),
+                enabled,
+            )
         }
     }
 }