Skip to content

Feat: Week9/nonshaman-m2 - #87

Open
nonshaman wants to merge 35 commits into
mainfrom
week9/nonshaman-m2
Open

Feat: Week9/nonshaman-m2#87
nonshaman wants to merge 35 commits into
mainfrom
week9/nonshaman-m2

Conversation

@nonshaman

Copy link
Copy Markdown
Collaborator

📚 주차 / 미션

  • 9주차 2번째 미션

📌 작업 내용

  • Modal Slice를 활용하여 전역으로 관리

📸 스크린샷

image

✅ 체크리스트

  • [o] 기능 정상 작동 확인
  • [o] 불필요한 주석 삭제
  • [o] 해당 주차 키워드 내용 이해

nonshaman and others added 30 commits April 12, 2026 20:51
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
…an-m1

Revert 83 revert 37 week4/nonshaman m1

@yewon20804 yewon20804 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

9주차 미션 2도 수고하셨습니다 ! 이번에는 Modal Slice 활용하여, 모달 기능을 추가해보는거였는데요 ! 잘 작성해주셨고 아래 코멘트 확인 부탁드립니다 ☺️


1. week9/src/slices/modalSlice.ts — export default 불필요한 변수 선언

// 이전
const modalReducer = modalSlice.reducer;
export default modalReducer;

// 수정
export default modalSlice.reducer;

변수에 담지 않고 carSlice.ts처럼 직접 export하셔도 됩니다 ! 그리고 이 방식이 더 일방적입니다 !


2. week9/src/components/Modal.tsx — 인라인 스타일 사용

현재 배경 오버레이에만 인라인 스타일을 사용하고, 나머지는 tailwind로 작성해주셨기 때문에 배경 오베리이도 Tailwind로 통일하면 좋을 것 같습니다 ☺️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants