[FIX] 투두 툴바 서브태스크 아이콘 표시 오류 및 드래그 리스트 자동 스크롤 수정#277
Conversation
- 아이콘이 나타내는 실제 의미(서브태스크 유무)에 맞게 prop 이름을 hasSubTask로 수정했습니다
- 생성/상세 모달에서 hasSubTask 값을 메모가 아닌 서브태스크 입력 존재 여부로 계산하도록 수정했습니다 - 오늘 화면 투두 카드에서도 이름이 바뀐 hasSubTask prop을 그대로 전달하도록 수정했습니다
- 세로 정렬 드래그 중 리스트 끝 근처에서 자동 스크롤되도록 autoScroll 옵션을 추가했습니다
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (5)
WalkthroughTodoToolbar의 메모 기반 플래그를 서브태스크 기반 플래그로 변경하고, 드래그 중 리스트 끝부분의 자동 스크롤 threshold를 설정했습니다. Changes투두 툴바 서브태스크 표시
드래그 자동 스크롤
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested labels: Suggested reviewers: ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Timo Performance ReportBundle Size — timo-web
Lighthouse — timo-web
Image Optimization — timo-web
측정 커밋: |
ISSUE 🔗
close #276
What is this PR? 🔍
투두 툴바에서 서브태스크 유무 아이콘이 실제로는 메모 입력 여부로 표시되던 문제를 수정하고, 투두 리스트 드래그 정렬 시 자동 스크롤이 동작하지 않던 문제를 수정했습니다.
배경
TodoToolbar의hasMemoprop이 아이콘 표시 여부를 결정했지만, 실제로는 서브태스크 유무를 나타내기 위한 아이콘이었습니다.hasMemo가 메모 텍스트 존재 여부로 계산되어 전달되면서, 서브태스크가 있어도 아이콘이 켜지지 않는 등 아이콘과 실제 데이터가 어긋났습니다.hasSubTask로 변경하고, 각 호출부에서 서브태스크 입력 존재 여부를 기준으로 값을 계산하도록 수정했습니다.투두 툴바 서브태스크 아이콘
TodoToolbar의hasMemoprop을hasSubTask로 변경하고, 생성/상세 모달·오늘 화면 카드에서 서브태스크 존재 여부 기준으로 값을 전달하도록 수정했습니다.CreateTodoModalContent와DetailTodoModalContent에서 각각subtaskField.subtaskInputs/detailTodoForm.subtaskInputs를 순회해 값이 있는 입력이 하나라도 있는지(.some(...))로hasSubTask를 계산합니다.TodayTodoCard는 이미 계산된toolbar.hasSubtask를 이름이 바뀐 prop으로 그대로 전달하도록만 수정했습니다.투두 리스트 드래그 자동 스크롤
DndSortableListProvider의DndContext에autoScroll={{ threshold: { x: 0, y: 0.2 } }}옵션을 추가했습니다.To Reviewers
hasSubTask계산에.some()을 쓴 방식이 기존 서브태스크 데이터 구조와 맞는지 확인 부탁드립니다.autoScroll의 y 임계값(0.2)은 초기값으로 넣었으며, 실제 스크롤 체감이 다르면 조정이 필요할 수 있습니다.
Screenshot 📷
Test Checklist ✔
pnpm check-types통과pnpm lint통과