PrezelFloatingButton 구현

[moondev03]

📌 작업 내용

• Modifier.dropShadowCache() 구현
• PrezelFloatingButton 구현
• PrezelFloatingMenuButton, PrezelFloatingButtonMenu, PrezelFloatingButtomMenuItem 구현

PrezelFloatingButtonStyle은 하위 컴포넌트인 PrezelFloatingButtonMenuItem의 디자인을 결정하는 역할을 합니다.

PrezelFloatingButtonMenu에서는 CompositionLocalProvider를 통해 현재 PrezelFloatingButtonStyle에 매핑된 PrezelFloatingButtonMenuItemStyle을 하위 컴포넌트로 전달하며, PrezelFloatingButtonMenuItem은 상위에서 제공된 스타일을 기준으로 자신의 UI를 구성합니다.

🧩 관련 이슈

• closed PrezelFloatingButton 구현 #23

📸 스크린샷

비고

PrezelFloatingButton 예시 코드

메뉴 기능이 없는 버튼입니다.

@Composable
private fun PrezelFloatingButtonSample() {
    PrezelFloatingButton(
        iconSource = DrawableIcon(resId = PrezelIcons.Blank),
        style = PrezelFloatingButtonStyle(),
        onClick = {},
    )
}

PrezelFloatingMenuButtom 예시 코드

메뉴 기능이 있는 버튼입니다.

@Composable
private fun PrezelFloatingMenuButtonSample() {
    var isExpanded by remember { mutableStateOf(false) }

    PrezelFloatingMenuButton(
        iconSource = DrawableIcon(resId = PrezelIcons.Blank),
        isExpanded = isExpanded,
        onClick = { isExpanded = !isExpanded },
        style = PrezelFloatingButtonStyle(),
    ) {
        PrezelFloatingButtonMenuItem(label = "A", onClick = {}, iconSource = DrawableIcon(resId = PrezelIcons.Blank))
        PrezelFloatingButtonMenuItem(label = "B", onClick = {}, iconSource = DrawableIcon(resId = PrezelIcons.Blank))
        PrezelFloatingButtonMenuItem(label = "C", onClick = {}, iconSource = DrawableIcon(resId = PrezelIcons.Blank))
    }
}

Summary by CodeRabbit

• 새로운 기능

  • 플로팅 버튼 컴포넌트 추가(프라이머리/세컨더리, 정규/소형) 및 풀폭 메뉴 항목 지원.
  • 확장 가능한 플로팅 메뉴 추가로 메인 버튼에서 컨텍스트 메뉴 표시 가능.
  • 드롭 섀도우 효과(시각적 그림자) 지원 추가.

• 기타

  • 아이콘 접근성 설명 방식 변경(리소스 기반) 및 닫기 문자열 리소스 추가.
  • 정적 분석 설정에 명명 규칙 항목 추가(비활성).

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

[coderabbitai]

Walkthrough

플로팅 버튼 컴포넌트와 관련 스타일 유틸리티를 추가합니다. 공개 컴포저블로 PrezelFloatingButton, PrezelFloatingMenuButton, PrezelFloatingButtonMenuItem 및 관련 스타일 열거형(크기·계층)과 데이터 클래스가 도입되었고, 메뉴 항목 전용 스타일 헬퍼(패딩·아이콘 크기 등)와 Modifier.dropShadowCache 유틸이 추가되었습니다. IconSource 인터페이스의 contentDescription이 프로퍼티에서 @Composable 함수로 변경되었고, DrawableIcon이 리소스 기반 설명을 반환하도록 업데이트되며 관련 문자열 리소스와 detekt 설정 항목이 추가되었습니다.

Possibly related PRs

• Team-Prezel/Prezel-Android PR 25: IconSource 및 버튼 컴포넌트 관련 API 변경을 도입한 PR로, 본 PR의 contentDescription API 리팩터 및 버튼 사용부와 코드 수준에서 직접 연관됩니다.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ 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%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목 'PrezelFloatingButton 구현'은 PR의 주요 변경 사항인 PrezelFloatingButton 컴포넌트 구현을 명확하게 요약하고 있습니다.
Description check	✅ Passed	PR 설명이 모든 필수 섹션(작업 내용, 관련 이슈, 스크린샷, 비고)을 포함하고 있으며, 구현 내용과 사용 예시가 상세히 작성되어 있습니다.
Linked Issues check	✅ Passed	PR의 모든 주요 변경사항(Modifier.dropShadowCache, PrezelFloatingButton, 메뉴 컴포넌트들, 스타일 구현)이 이슈 #23의 목표를 충족하고 있습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 이슈 #23의 요구사항과 직접적으로 관련되어 있으며, 범위를 벗어난 변경은 없습니다.

_{✏️ 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/floating/PrezelFloatingButton.kt`:
- Around line 20-41: The Icon's contentDescription can be null which breaks
accessibility; update PrezelFloatingButton to require a non-null
contentDescription (add a contentDescription: String parameter to
PrezelFloatingButton and pass it to Icon) or alternatively make
IconSource.contentDescription non-null across implementations (e.g., change
IconSource.contentDescription to String and update DrawableIcon), but choose one
approach and update all call sites so Icon(contentDescription =
contentDescription) always receives a non-null value (referencing
PrezelFloatingButton, Icon, IconSource.contentDescription, and DrawableIcon).

In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/button/floating/PrezelFloatingButtonStyle.kt`:
- Around line 73-81: The PRIMARY branch in prezelFloatingMenuButtonContentColor
currently uses a hardcoded PrezelColorScheme.Light.textMedium which bypasses
active theme palettes; change the PRIMARY case to use the passed-in
colors.textMedium (the same source used for SECONDARY) so the function respects
the current PrezelTheme colors instead of the fixed Light scheme.

In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/button/floating/PrezelFloatingMenuButton.kt`:
- Around line 65-81: The hardcoded Korean label for the close icon should be
moved to a string resource and accessed via stringResource: add an entry in
designsystem's res/values/strings.xml (e.g. name="floating_button_close"), then
update PrezelMainFloatingButton to call
stringResource(R.string.floating_button_close) and pass that value to
DrawableIcon(..., contentDescription = ...); reference the
PrezelMainFloatingButton function, the currentIconSource assignment, and
PrezelIcons.Cancel when making this change and ensure the necessary
androidx.compose.ui.res.stringResource import is added.

🧹 Nitpick comments (2)

Prezel/detekt-config.yml (1)

50-51: 규칙 전체 비활성화 대신 더 세밀한 설정 고려

MatchingDeclarationName 규칙을 전체적으로 비활성화하면 코드베이스 전반에서 파일명과 선언명 간의 일관성이 저하될 수 있습니다. 특정 파일이나 디렉토리에서만 예외를 허용하는 것이 더 안전한 접근입니다.

♻️ 대안: excludes를 사용한 세밀한 설정

     MatchingDeclarationName:
-        active: false
+        active: true
+        excludes:
+            - '**/floatingbutton/**'

또는 해당 파일들을 규칙에 맞게 재구성하는 방법도 고려해 보세요.

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

55-77: 프리뷰 중복 간소화 고려

프리뷰에서 동일 패턴의 호출이 반복되어 리스트로 묶으면 가독성이 좋아집니다.

♻️ 예시 리팩토링

         ) {
-            PrezelFloatingButton(
-                iconSource = DrawableIcon(resId = PrezelIcons.Blank),
-                style = PrezelFloatingButtonStyle(hierarchy = PrezelFloatingButtonHierarchy.PRIMARY, size = PrezelFloatingButtonSize.REGULAR),
-                onClick = {},
-            )
-
-            PrezelFloatingButton(
-                iconSource = DrawableIcon(resId = PrezelIcons.Blank),
-                style = PrezelFloatingButtonStyle(hierarchy = PrezelFloatingButtonHierarchy.PRIMARY, size = PrezelFloatingButtonSize.SMALL),
-                onClick = {},
-            )
-
-            PrezelFloatingButton(
-                iconSource = DrawableIcon(resId = PrezelIcons.Blank),
-                style = PrezelFloatingButtonStyle(hierarchy = PrezelFloatingButtonHierarchy.SECONDARY, size = PrezelFloatingButtonSize.REGULAR),
-                onClick = {},
-            )
-
-            PrezelFloatingButton(
-                iconSource = DrawableIcon(resId = PrezelIcons.Blank),
-                style = PrezelFloatingButtonStyle(hierarchy = PrezelFloatingButtonHierarchy.SECONDARY, size = PrezelFloatingButtonSize.SMALL),
-                onClick = {},
-            )
+            listOf(
+                PrezelFloatingButtonStyle(PrezelFloatingButtonHierarchy.PRIMARY, PrezelFloatingButtonSize.REGULAR),
+                PrezelFloatingButtonStyle(PrezelFloatingButtonHierarchy.PRIMARY, PrezelFloatingButtonSize.SMALL),
+                PrezelFloatingButtonStyle(PrezelFloatingButtonHierarchy.SECONDARY, PrezelFloatingButtonSize.REGULAR),
+                PrezelFloatingButtonStyle(PrezelFloatingButtonHierarchy.SECONDARY, PrezelFloatingButtonSize.SMALL),
+            ).forEach { style ->
+                PrezelFloatingButton(
+                    iconSource = DrawableIcon(resId = PrezelIcons.Blank),
+                    style = style,
+                    onClick = {},
+                )
+            }
         }

[HamBeomJoon]

고생하셨습니당

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/button/floating/PrezelFloatingMenuButton.kt`:
- Around line 212-227: In PreviewFloatingMenuButton the local variable
iconSource is created but not used; update the PrezelFloatingMenuButton(...)
call to pass iconSource (the variable) into the iconSource parameter instead of
constructing a new DrawableIcon, or if the variable is unnecessary remove its
declaration; refer to PreviewFloatingMenuButton, the iconSource variable, the
PrezelFloatingMenuButton(...) call and the PrezelFloatingButtonMenuItem usages
when making the change.

♻️ Duplicate comments (1)

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

74-75: 문자열 리소스 사용으로 로컬라이징 지원 - 이전 리뷰 의견 해결됨

하드코딩된 한국어 텍스트 "플로팅 버튼 닫기"가 R.string.close_floating_btn_content_desc 리소스로 올바르게 외부화되었습니다.

🧹 Nitpick comments (3)

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

20-43: 접근성 개선 확인 - 하지만 여전히 nullable입니다.

IconSource.contentDescription()이 이제 @Composable 함수로 변경되어 문자열 리소스를 통한 로컬라이징을 지원합니다. 그러나 반환 타입이 여전히 String?이므로 contentDescTextId를 제공하지 않으면 스크린 리더에서 버튼 설명이 누락될 수 있습니다.

호출 측에서 접근성을 위해 contentDescTextId를 반드시 제공하도록 문서화하거나, 향후 필수 매개변수로 변경하는 것을 고려해 주세요.

Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/icon/IconSource.kt (1)

27-29: contentDescTextId 참조 시 it 대신 명시적 변수 사용 권장

현재 구현이 정확하지만, let 블록 내에서 it 대신 contentDescTextId를 직접 참조하고 있습니다. 이는 동작에 문제가 없으나, 일관성을 위해 it을 사용하거나 명시적 람다 파라미터를 사용하는 것이 좋습니다.

♻️ 제안된 변경

     `@Composable`
-    override fun contentDescription(): String? = contentDescTextId?.let { stringResource(contentDescTextId) }
+    override fun contentDescription(): String? = contentDescTextId?.let { stringResource(it) }

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

186-188: 파라미터와 동일한 이름의 변수 섀도잉

isExpanded 파라미터를 동일한 이름의 var로 섀도잉하고 있습니다. 코드가 동작하지만, 파라미터와 로컬 상태를 구분하기 어려워 가독성이 떨어집니다.

♻️ 변수명 변경 제안

 `@Composable`
 private fun PreviewFloatingMenuButton(
-    isExpanded: Boolean,
+    initialExpanded: Boolean,
     style: PrezelFloatingButtonStyle,
 ) {
-    var isExpanded by remember { mutableStateOf(isExpanded) }
+    var isExpanded by remember { mutableStateOf(initialExpanded) }

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/icon/IconSource.kt`:
- Line 29: contentDescription 함수의 현재 구현 passes the args array as a single
parameter to stringResource, so format placeholders aren't filled; change the
call in IconSource.contentDescription to pass the vararg elements using the
spread operator (*) when forwarding args (i.e., use stringResource(resId,
*args)) so contentDescTextId and stringResource receive the vararg values
correctly.

[HamBeomJoon]

👍