Skip to content

곰 3주차 미션 #4

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

곰 3주차 미션 #4

wants to merge 17 commits into from

Conversation

gomx3
Copy link

@gomx3 gomx3 commented Oct 9, 2024

3주차 미션 일부 완성했습니다.

  • navbar 및 sidebar 구현
  • 카테고리 페이지 구현
  • 영화 데이터 받아오기
  • 영화 포스터 및 목록 보여주는 부분이 어색 -> 보완 필요

@juuuuone
Copy link
Collaborator

너무 잘 구현해주셨어요~!! 다만 todolist에서 id를 랜덤한 번호보다는 이것도 useState이용해서 삭제/추가될때마다 핸들링해주시면 좋을것같아요~~
css는 외부파일사용보다는 인라인스타일이나 Styled Components 이용하시는게 프로젝트 커질때 훨씬 다루기 쉬울거예요!!

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