Sehun week2 task

[seulnan]

전체적인 피드백

1. 웬만한건 멘토링 끝나고 다 해준것같은데..다시강조하자면 어려운 부분은 지피티 외에도 오픈소스 (다른친구들 코드, 프롬프트) 적극이용합시다! 그게 개발속도를 빠르게 올림과 동시에 성장을 빠르게 할 수 있어용(가장 중요한 코드읽고 커스텀하는 능력향상에 직빵)

layout

1. 별은 svg파일로 넣는게 좋겠죠?참고로 svg파일은 기본값이 인라인요소입니다!
2. 전체적으로 배경이미지의 부재, review이미지부재, 폰트색깔 미적용 등...지피티와 오픈소스많이 이용해서 뚝딱 완성해냅시다!

recipe

1. hr 태그를 쓰다니...태그에 대한 이해가 굉장히 높군요,,,이런 html 기본태그는 간단한 페이지(우리과제)같은거에서 쓰면 코드가독성도 올라가고 가벼운 커스텀은 가능하지만, 위치조정이 사알짝 어렵다는 단점이 있으니 참고!
2. 위와 겹치는 내용인데 ul,ol,li태그를 이용해서 살짝만 커스텀하면 p태그끼리 충돌하는걸 방지가능!(개발자도구로 보니까 엄청난 취소선이...겹치는 스타일이 많아보이네용..)
3. 부모자식 모두 p태그를 이용해서 구현했던데 보통 부모태그는 div을 이용하고 그 속 텍스트만 있는 태그에 p를 쓰는 경우가 많습니다!
4. instructions, nutrition 부분의 여백값이 너무 큰것같네용 전체적으로 가장 큰 부모태그인 card와 그 아래 부모태그인 instructions에 gap을 많이 줘서인것같습니다!

[hubluish]

피드백 남기겠습니다.
인상깊은 것

1. 태그 이름을 다르게 해서 알아보기 쉬웠다
2. 클래스 작명법이 탁월한 것 같아요

고칠것

1. review 이미지 크기가 다름 - css 수정
2. 박스 크기가 가로 세로로 너무 김 - width height 데브모드 보고 정확히 지정 가능
3. 이후 박스 왼쪽 마진을 좀 줘서 텍스트와 맞춰야 할 것 같음
4. body를 감싸는 container를 따로 만들었는데 굳이? 싶음 바로 body에 크기 및 여타 css 적용해도 될 듯
5. background 적용 안됨 - absolute 쓰면 편함
6. 미디어쿼리 상태에서 제목, 부제, 박스 안 텍스트, 박스크기, body 사이즈 기타등등 에 스타일이 적용 안됨 - 적용만 하면 배열은 알맞아서 피그마 모양대로 바뀔듯

[hubluish]

코드리뷰 남기겠습니다!
점점 발전하는 모습이 보기 좋습니다! 전부터 그랬지만 클래스 명 짓는 것도 기가 막히고, 구조도 알아보기 쉽게 잘 짜여 있는 것 같아요! css도 필요없거나 겹치는 건 딱히 보이지 않고, 적절해 보입니다. js 코드도 주석을 잘 달아 주어서 보기 편리하였습니다.
거의 완벽한 완성도를 가졌지만, 몇 가지 꼽아보자면

1. input-container 안에 있는 원도 클릭하면 체크박스가 표시되는 문제가 있습니다.
2. input-container 안의 원에 왼쪽 마진을 조금 더 주면 좋을 것 같습니다.
3. 어둠 모드에서 버튼의 원본, hover했을 때, 클릭했을 때 색이 다릅니다.
4. 항목을 클릭했을 때 텍스트가 오른쪽으로 근소하게 움직이는 사소한 버그가 있습니다 그러나 크게 신경쓰이진 않음!
   이정도 있는 것 같습니다. 좋은 하루 되세요!