Skip to content

박영신 week1#12

Merged
bk-git-hub merged 7 commits intoidid10from
idid10-week1
Mar 19, 2026
Merged

박영신 week1#12
bk-git-hub merged 7 commits intoidid10from
idid10-week1

Conversation

@idid10
Copy link
Copy Markdown

@idid10 idid10 commented Mar 18, 2026

✅ 제출 정보

  • Week: Week <01>

✅ 체크리스트 (필수)

  • PR의 base branchmain이 아니라 내 GitHub 핸들 브랜치(<handle>)다
  • compare branch가 <handle>-week-xx 형식이다
  • 변경 사항이 이번 주 과제 범위에 해당한다
  • (필요 시) 실행 방법을 적었다
  • (가능하면) 결과 스크린샷/데모 링크를 첨부했다

🧩 구현 내용 요약

  • index.css 내 전역 변수를 활용하여 Colors 및 Typography 정의
  • Figma 가이드에 명시된 Primary, Success, Text 색상 및 Heading, Body 폰트 수치 반영
  • TodoCard, TodoHeader, TodoList 컴포넌트 분리
  • 할 일 데이터를 별도의 constants 파일로 분리
image

❓ 궁금한 점

@bk-git-hub bk-git-hub changed the title Week: Week <01> 박영신 week1 Mar 19, 2026
Copy link
Copy Markdown
Collaborator

@bk-git-hub bk-git-hub left a comment

Choose a reason for hiding this comment

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

영신님 수고 많으셨습니다

전반적으로 잘 진행해주셨고 구현도 잘 해주셨습니다.
사용하지 않는 CSS 파일들만 가볍게 정리해주시면 더 깔끔해질 것 같습니다!

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.

index.cssApp.css에 vite 템플릿으로 생성된 사용하지 않는 스타일들은 미리 정리하면 좋을 것 같아요. 과제와 관련 없는 스타일은 정리해두면 css구조가 훨씬 명확해질 것 같아요

@bk-git-hub bk-git-hub merged commit 4e7df97 into idid10 Mar 19, 2026
2 checks passed
@bk-git-hub bk-git-hub deleted the idid10-week1 branch March 19, 2026 01:16
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