Skip to content

Suminn01 week01#8

Merged
bk-git-hub merged 2 commits intosuminn01from
suminn01-week01
Mar 18, 2026
Merged

Suminn01 week01#8
bk-git-hub merged 2 commits intosuminn01from
suminn01-week01

Conversation

@suminn01
Copy link
Copy Markdown

@suminn01 suminn01 commented Mar 17, 2026

✅ 제출 정보

  • Week: Week <01>

✅ 체크리스트 (필수)

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

🧩 구현 내용 요약

  • 피그마 기준으로 레이아웃 구현
  • 컴포넌트 분리
image

❓ 궁금한 점

  • 없습니다.

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도 역할별로 잘 나눠주셔서 구조가 보기 좋았습니다. TypeScript 기반으로 프로젝트를 시작하신 점도 좋았습니다.

이번에는 initial commit에 모든 작업사항이 다 묶여있었는데, 다음부터는 작업 단위에 맞게 조금 더 나누어 작성해주시면 좋을 것 같습니다 :)

커밋 안내 문서 링크
https://www.notion.so/Git-Commit-Guide-327793efb80180bcb8f9c357c580a48d

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.

현재 src/index.css 에 Vite 기본 템플릿 스타일이 많이 남아 있어서, 실제 과제용 스타일 App.css이랑 전역에서 서로 섞여 적용되고 있습니다. 특히 :root, #root, heading, dark mode 관련 스타일은 과제 UI와 직접 관련이 없어서 정리해주시면 스타일 구조가 더 명확해지고 스타일 충돌 위험이 줄어들 것 같아요!

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.

현재 할 일 목록을 div로 감싸고 있는데, 의미상 리스트이기 때문에 ul / li 구조로 작성하면 조금 더 시맨틱한 마크업이 될 것 같아요

@bk-git-hub bk-git-hub merged commit a2cd741 into suminn01 Mar 18, 2026
3 checks passed
@bk-git-hub bk-git-hub deleted the suminn01-week01 branch March 18, 2026 07:23
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