PrezelCheckbox 구현

[HamBeomJoon]

📌 작업 내용

• PrezelCheckbox 컴포저블 추가
  • size 옵션: REGULAR, LARGE
  • checked 상태 외부에서 관리 (stateless)
  • 클릭 시 onCheckedChange 호출
  • 접근성: contentDescription 및 Role.Checkbox 적용

• Preview에서도 클릭시 상태 변경가능하도록 했습니다.

🧩 관련 이슈

• close PrezelCheckbox 구현 #31

📸 스크린샷

📢 논의하고 싶은 내용

contentDescription, semantic이 맞나요

Summary by CodeRabbit

• 새로운 기능

  • 체크박스 UI 컴포넌트 추가(일반·큰 사이즈) 및 테마 미리보기 제공

• 개선사항

  • 스크린리더용 접근성 설명 추가(체크박스 문자열 리소스)
  • 플로팅 버튼의 확장 상태 설명 문자열 키 정리로 일관성 향상

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

[coderabbitai]

Walkthrough

이 PR은 디자인 시스템에 PrezelCheckbox 컴포저블을 추가하고 문자열 리소스를 정리합니다. 공개 함수 PrezelCheckbox(checked: Boolean, modifier: Modifier = Modifier, size: CheckboxSize = CheckboxSize.REGULAR, onCheckedChange: (Boolean) -> Unit)와 enum CheckboxSize(REGULAR, LARGE)를 추가했습니다. 체크 상태에 따라 CheckCircleFilled/CheckCircleOutlined 아이콘과 테마 색상이 적용되며 48.dp 터치 대상과 크기별 아이콘(24.dp/32.dp)을 사용하고 Role.Checkbox 의미론을 설정합니다. 미리보기 컴포저블도 추가되었고, PrezelFloatingMenuButton의 닫기 콘텐츠 설명 키를 변경하며 strings.xml에 core_designsystem_checkbox_desc를 추가했습니다.

Possibly related PRs

• PrezelFloatingButton 구현 #37: PrezelFloatingMenuButton.kt와 관련 문자열 리소스 키(close_floating_btn_content_desc → core_designsystem_close_floating_btn_content_desc)를 변경한 커밋이 있어 코드 수준에서 직접적인 연관이 있습니다.

🚥 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	The title "PrezelCheckbox 구현" clearly and concisely summarizes the main change - implementing the PrezelCheckbox component.
Description check	✅ Passed	The description follows the template structure with all required sections (작업 내용, 관련 이슈, 스크린샷) properly filled, providing clear implementation details and addressing accessibility concerns.
Linked Issues check	✅ Passed	The PR successfully implements the PrezelCheckbox component (#31) with theme-aligned design, proper size options, stateless API, and accessibility features (Role.Checkbox, contentDescription) as required.
Out of Scope Changes check	✅ Passed	All changes are directly related to PrezelCheckbox implementation (#31). String resource updates support the checkbox feature, and the floating button change aligns with design system resource naming consistency.

_{✏️ 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/PrezelCheckbox.kt`:
- Around line 93-108: The preview uses empty contentDescription values in
CheckboxRowPreview for PrezelCheckbox which is inappropriate; update the
contentDescription arguments in the Preview composables (the CheckboxRowPreview
block and the similar block around lines 126-138) to meaningful, non-empty
strings (e.g., "Regular checkbox (checked)" and "Regular checkbox (unchecked)"
or other concise descriptions) so that PrezelCheckbox receives proper
accessibility text for both states.
- Around line 64-72: The Box uses .semantics { role = Role.Checkbox } with
.clickable which doesn't expose checked state to accessibility; replace the
clickable usage with the toggleable modifier so the system announces
checked/unchecked automatically. Update the modifier chain in PrezelCheckbox
(the Box's modifier) to use androidx.compose.foundation.selection.toggleable
(preserving interactionSource, indication, and the onCheckedChange boolean flip)
and remove manual checkbox semantics so the checked parameter passed into
PrezelCheckbox is reflected by toggleable's checked argument.

In `@Prezel/core/designsystem/src/main/res/values/strings.xml`:
- Around line 3-4: Ensure the new string resource keys
core_designsystem_close_floating_btn_content_desc and
core_designsystem_checkbox_desc are added to all other locale strings.xml files
(e.g., values-*/strings.xml) so translations aren’t missing; for each locale,
add the same keys with the proper translated text or a placeholder translation
and keep the exact resource names to avoid runtime missing-resource errors.

[moondev03]

피그마에는 나와있지 않지만 보통 체크박스는 텍스트가 함께 사용될 것 같습니다.

이를 위한 별도의 컴포넌트를 구현하거나 해당 컴포넌트에서 처리하는게 좋을 것 같습니다.

참고로 예시 화면은 작업중인 UI 페이지에 있어요.