[Bugfix] 임시 일정/할 일 삭제 시 서버 요청 방지

[yeonjin719]

🔗 관련 이슈

• closes 🐞 [BugFix] 글자 정렬 깨짐 문제 해결 #120
• closes 🐞 [BugFix] 일정/투두 생성 전 삭제 시 에러 안뜨도록 수정 #124
• closes ♻️ [Refactor] 투두 삭제 버튼에 호버 시 마우스 커서 클릭 가능한 형태로 변경 #125

---

🧩 작업 요약 (TL;DR)

이 PR에서 무엇을 왜 했는지 한 줄로 요약해주세요.

• 일정/할 일 임시 데이터 삭제 시 서버 요청을 막고, 🐞 [BugFix] 글자 정렬 깨짐 문제 해결 #120 관련 카드 UI 정렬 및 커서 스타일을 보완했습니다.

---

🔄 변경 유형

해당되는 항목에 체크해주세요.

• ✨ Feature
• 🐞 Bug Fix
• 🔨 Refactor (기능 변화 없음)
• 🎨 UI / UX
• ⚙️ Setting / Infra
• 🧪 Test
• 📄 Docs

---

📌 주요 변경 사항

리뷰어가 집중해서 봐야 할 포인트

• CalendarEvent에 isTemporary 플래그를 추가해 서버 반영 전 로컬 임시 데이터와 서버 데이터를 구분했습니다.
• 임시 일정/할 일 삭제 시 delete API 요청 없이 로컬 데이터 제거 및 모달 닫기만 수행하도록 처리했습니다.
• 임시 항목에서 상세 조회, patch, 색상 변경 patch가 서버로 나가지 않도록 방어 조건을 추가했습니다.
• 임시 항목을 다시 열고 저장하는 경우 수정 요청이 아닌 생성 요청 흐름을 타도록 보완했습니다.
• 일정 상세 카드 텍스트 정렬을 flex-start로 맞췄습니다.
• 투두 카드 삭제 아이콘 hover 시 pointer 커서가 보이도록 스타일을 추가했습니다.

---

🖼️ 스크린샷 / 영상 (선택)

UI 변경이 있다면 꼭 첨부해주세요.

Before	After

---

🧠 리뷰 요청 포인트

리뷰어에게 특히 봐줬으면 하는 부분

• 로직 설계
• 상태 관리 방식
• 네이밍
• 성능 / 렌더링
• 기타: isTemporary 기준으로 서버 데이터와 로컬 임시 데이터를 분기하는 방식이 적절한지 확인 부탁드립니다.

---

⚠️ 체크리스트 (PR 올리기 전)

• 로컬에서 정상 동작 확인
• 기존 기능에 영향 없음
• 불필요한 console.log 제거
• 린트 / 타입 에러 없음
• 관련 이슈 연결 완료

---

🚧 미완 / 후속 작업

이 PR에서 다루지 않은 내용이나 추후 작업

• N/A

---

💬 기타 참고 사항

리뷰어가 알면 좋은 맥락, 트레이드오프, 고민 지점

• 기존 임시 이벤트 ID가 Date.now() 기반 양수라서 id > 0 조건만으로는 서버 저장 여부를 판단하기 어려웠습니다.
• 이를 해결하기 위해 isTemporary 플래그를 추가해 로컬 전용 항목을 명시적으로 구분했습니다.
• 삭제 흐름뿐 아니라 상세 조회, 수정, 색상 변경 patch에서도 임시 항목이 서버 요청을 발생시키지 않도록 함께 방어했습니다.
• npm run build 통과 확인했습니다.

@copilot 이 PR을 아래 기준으로 검토해주세요:

.github/instructions/capstone.instructions.md 파일을 지침으로 삼으세요.
폴더/파일 위치가 프로젝트 구조 규칙과 일치하는지
컴포넌트가 단일 책임 원칙(SRP)을 지키는지
import 방향이 올바른지 (shared → features 역방향

[Copilot]

Pull request overview

임시(로컬) 일정/할 일에 대해 isTemporary 플래그로 서버 반영 여부를 구분해, 삭제/상세조회/수정/색상변경 시 불필요한 서버 요청을 방지하고 일부 UI 스타일을 보완한 PR입니다.

Changes:

• CalendarEvent.isTemporary를 도입하고, 임시 항목인 경우 delete/detail/patch/색상 patch 요청이 나가지 않도록 조건 분기 추가
• 서버 생성 후 임시 이벤트를 실제 id로 교체할 때 isTemporary: false로 상태 전환
• 카드 UI 정렬 및 투두 삭제 아이콘 커서 스타일 보완

한 줄 요약

isTemporary 기반으로 “임시 로컬 데이터 vs 서버 데이터”를 분리해 서버 요청을 차단하는 방향은 타당하나, 투두 삭제 아이콘 클릭 요소의 키보드 접근성 보완이 필요합니다.

발견된 위반 항목

• src/features/Todo/components/TodoCard/TodoCard.tsx: 클릭 가능한 SVG 아이콘(Trash)에 대해 키보드 접근/스크린리더 라벨이 없어 접근성 문제가 남아있음(이번 변경 라인에 포함).

추천 구조

• (선택) shared/ui에 아이콘 버튼/액션 버튼 컴포넌트가 이미 있다면 그 컴포넌트로 교체하거나, 없다면 추후 아래처럼 공용화 고려:
  • src/shared/ui/common/IconButton/
    • IconButton.tsx
    • IconButton.style.ts

최소 리팩토링 단계

1. TodoCard의 Trash SVG에 role, tabIndex, aria-label, onKeyDown(Enter/Space)를 추가해 키보드로도 삭제 트리거 가능하게 수정
2. (선택) 클릭 액션을 SVG 직접 바인딩 대신 <button type="button"> 래핑 형태로 정리해 접근성/스타일 일관성 개선

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/shared/ui/Modals/TodoEditor/TodoEditorContent.tsx	투두 상세 hydration에 initialEvent 전달로 임시 여부 분기 지원
src/shared/types/calendar/types.ts	CalendarEvent에 isTemporary 플래그 추가
src/shared/hooks/form/useTodoEditorForm.ts	임시 투두는 patch 대신 create 흐름으로 전환
src/shared/hooks/addTodo/useTodoFooter.tsx	임시 투두 색상 변경 시 patch 요청 차단
src/shared/hooks/addTodo/useTodoDetailHydration.ts	임시 투두 상세 조회 요청 차단
src/shared/hooks/addSchedule/useScheduleSubmitFlow.ts	임시 일정은 편집 모드여도 create 흐름으로 전환
src/shared/hooks/addSchedule/useSchedulePatch.ts	임시 일정 patch 요청 차단
src/shared/hooks/addSchedule/useScheduleFooter.tsx	임시 일정 삭제/색상 patch 요청 차단 및 삭제 시 모달만 닫기
src/features/Todo/components/TodoCard/TodoCard.tsx	삭제 아이콘 커서 스타일 추가(UI/UX)
src/features/Calendar/utils/helpers/calendarPageHelpers.ts	로컬 임시 생성 이벤트에 isTemporary: true 설정
src/features/Calendar/hooks/useCalendarEvents.ts	서버 응답 후 id 교체 시 isTemporary: false로 전환
src/features/Calendar/hooks/useCalendarDraftEvent.ts	편집 중이더라도 임시 이벤트면 모달 닫을 때 로컬에서 제거
src/features/Calendar/components/EventDetailCard/EventDetailCard.style.ts	상세 카드 텍스트 좌측 정렬(flex-start) 보정