Eunseo week2 task

[seulnan]

전체적인 피드백

1. 일단 코드에서 나열심히 공부했어요가 너무 보여서...고생많았어요ㅠㅠㅠ
2. 파일명은 깨질 우려가 있으니 영어로 바꿔봅시다!
3. 폰트나 이미지는 assets폴더로 따로 분리하는 경우가 많아용!
4. 커밋 작은 단위로 한거 너무 잘했어요 커밋 형식만 "[FIX] : 이런거 수정"로 통일해주면 완벽해요!

layout

1. 아주 잘한점: 시맨틱태그 section을 썼다는 점에서 박수 오만개드립니다
2. 주석도 너무 잘 달았어요!!
3. margin 좌우로 auto를 하면 자동으로 중앙정렬되는 점을 이용해보는건 어떨까용?

recipe

1. 백그라운드를 넣는걸 어려워하는것같은데 다른사람코드를 참고해보면서 수정하면 좋아용! 잘짜인 코드를 알맞게 활용하는것도 중요해용
2. span,div, p태그에 대해 아주 자세히 공부한 티가 납니당 좋아용 ! 다만, 텍스트때문에 p태그로 넣은것같은데 이 부분에서는 한줄단위로 다루면서 여백을 조절하는게 편하니 div로 바꿔보는건 어떨까용?
3. 위 내용과 살짝 연관되는 내용인데 br태그는 사실 잘 쓰지 않습니다..올드한 느낌! inline인 p태그 대신 div3개로 분리해서 자동으로 줄바꿈이 되도록 해보는건 어때용?
4. span, p태그는 인라인요소라 flex에서 다루기 살짝 어렵습니다. div가 flex에서는 좀더 많이 유연하고 잘 이용되니 div를 적극이용해봐용! flex로 반응형디자인할때 유리해집시다!

[xxoyya]

수고하셨습니다!!😍

1. width가 738px이 됐을 때 바뀌게 되는데 바뀌었을 때, 화면의 중앙에 위치하지 않아요. body에 스타일을 주어 중앙에 정렬되도록 만들어 보는 건 어떨까용?

2. 반응형 전 기본 페이지에서 리뷰박스의 위치가 너무 내려가 있는데 star-box-1의 스타일 값에서 margin-top이 200px로 되어 있어서 그런 것 같아요. margin-top 값을 조금 줄여도 좋을 것 같습니다!

3. 타이틀에 text-align 으로 center에 올 수 있도록 만드려고 한 것 같은데 적용이 되지 않는 것 같아요. 그 이유가 html 코드를 작성할 때 h1에 style을 바로 text-align: left라고 작성하여 반응형을 했을 때 center가 적용이 되지 않는 것 같습니다. h1에 바로 style을 주지 않고 따로 클래스를 만들어서 left 스타일을 주는 것도 좋을 것 같아용!

[xxoyya]

전체적으로 코드를 매우x100 잘 짜신 것 같아요!!

각 todo item에 드래그 했을 때 색이 변하는 기능 추가한 것과 완료된 todo들이 자동적으로 가장 위로 올라가는 기능도 좋은 것 같아용❤️
이번 과제에서는 딱히 수정해야 할 부분이 보이지 않았어요!

수고하셨습니다!!