Skip to content

[FIX#81] 투두 리스트 컴포넌트 터치 영역 수정#82

Merged
nhyeonii merged 1 commit intodevelopfrom
fix/#81-todo-component-clickable
Jan 18, 2026
Merged

[FIX#81] 투두 리스트 컴포넌트 터치 영역 수정#82
nhyeonii merged 1 commit intodevelopfrom
fix/#81-todo-component-clickable

Conversation

@nhyeonii
Copy link
Copy Markdown
Contributor

@nhyeonii nhyeonii commented Jan 18, 2026

Related issue 🛠

Work Description ✏️

  • 투두 리스트 컴포넌트 터치 영역 수정

Screenshot 📸

2026-01-18.8.31.44.mov

Uncompleted Tasks 😅

  • N/A

To Reviewers 📢

터치 영역 수정햇슴 !!

Summary by CodeRabbit

릴리스 노트

  • 버그 수정
    • 체크리스트 항목의 상호작용 개선: 전체 행을 클릭하여 체크리스트 항목을 선택할 수 있도록 업데이트되었습니다.

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

@nhyeonii nhyeonii self-assigned this Jan 18, 2026
@nhyeonii nhyeonii requested a review from a team as a code owner January 18, 2026 11:32
@nhyeonii nhyeonii added FIX🐛 오류 및 버그 수정 나현🍒 나현 담당 labels Jan 18, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 18, 2026

Walkthrough

ChallengeChecklist 컴포넌트의 클릭 이벤트 처리 방식을 변경했습니다. Icon 아이콘의 개별 클릭 처리를 제거하고, 전체 Row 컨테이너에 클릭 이벤트를 적용하여 터치 영역을 개선했습니다.

Changes

응집군 / 파일(들) 변경 요약
클릭 이벤트 리팩토링
app/src/main/java/com/cherrish/android/presentation/challenge/component/ChallengeChecklist.kt
ChallengeChecklistItem에서 onChecklistClick 파라미터 제거 및 Icon의 clickable 수정자 삭제. Row 컨테이너에 noRippleClickable을 통해 클릭 이벤트 통합 적용

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3분

🚥 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 제목은 풀 리퀘스트의 주요 변경사항인 투두 리스트 컴포넌트의 터치 영역 수정을 명확하게 설명합니다.
Description check ✅ Passed PR 설명이 리포지토리의 템플릿 구조를 따르고 있으며, 관련 이슈, 작업 내용, 스크린샷, 검토자에게 전달할 메시지가 포함되어 있습니다.
Linked Issues check ✅ Passed 코드 변경사항이 issue #81의 목표인 TODO 컴포넌트의 터치 영역 수정 요구사항을 충족합니다. Row 컨테이너에 clickable 동작을 추가하고 개별 아이콘 클릭 동작을 제거하여 터치 영역을 명확히 했습니다.
Out of Scope Changes check ✅ Passed 변경사항은 ChallengeChecklist.kt 파일의 터치 영역 수정에만 집중하고 있으며, 연관된 issue #81의 범위 내에 있습니다.

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

✨ Finishing touches
  • 📝 Generate docstrings

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.

❤️ Share

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

Copy link
Copy Markdown
Contributor

@hyeminililo hyeminililo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

감사합니다 ~~~!!! 수고하셨습니다아 ~!

Copy link
Copy Markdown
Contributor

@sohee6989 sohee6989 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습ㄴ디ㅏ!!

@nhyeonii nhyeonii merged commit 27b0c8e into develop Jan 18, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FIX🐛 오류 및 버그 수정 나현🍒 나현 담당

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] TO-DO 컴포넌트 터치 영역 수정

3 participants