Next 박창기

[p-changki]

요구사항

기본

• [x]
• []
• []

심화

• [x]
• []

주요 변경사항

스크린샷

멘토에게

• 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

[devbini]

🎉 고생하셨습니다

제 노트북 이슈로 올려주신 Files changed에서 파일을 클릭하면 튕겨버리는 브라우저 저주에 걸려서... 통합 리뷰 아래로 쭉 써 두도록 할게요.

PR 올려주신 내용에 아무 설명이 없어서 어디가 구현이 안 된 건지 직접 찾아보고 있습니다만... 댓글이나 글쓰기 등 API 부분은 거의 미 개발상태로 인지하고 있습니다.
그래서, 미구현 된 기능 제외하고 코드를 확인했어요.

먼저.. 테일윈드를 사용한 모습은 꽤나 인상적입니다. 아직 진도가 안 나간거로 아는데, 독학하느라 정말 고생하셨을 것 같아요.

전체적으로 디자인 구성은 잘 되었는데, 반응형이 아니라 px로 되어있는 부분이 몇몇 컴포넌트에서 보여서.. 고정된 픽셀값은 수정 해 주는 게 좋아 보입니다.

이어서.. 클릭 이벤트가 발생하는 HTML 태그는 가능한 <button>으로 해 주시는 게 좋습니다. 태그 규칙의 일관성도 그렇고, SEO와 사용성에 있어서도 이익이 있어요! 지금은 <li>로 되어있는 부분이 몇 군데 있더라구요.

사실 이런 사소한 최적화 이슈 말고, 기능 상 문제가 될 부분은 없어 보이더라구요.
프로젝트 구조와 라우트 폴더 구조, api 호출시에 query 필터링을 적용한 점이나 일부 데이터 포매팅까지, 신경써서 코드를 만들어주신 게 느껴져서 기분이 좋았어요.

우선 가장 급한 건 반응형 스타일 적용만 잘 해주시고,
그 다음 API 구현(local db 내)으로 넘어가면, 자연스럽게 좋은 코드가 완성되지 않을까 싶습니다.

고생하셨어요! :D