총평
- 디자인이 깔끔하고 이쁩니다. 통일된 색상들이 유저로 하여금 좋은 느낌을 줍니다.
- 다만 뜬금없는 색상이 들어갈 때도 있네요. ex. 작성완료 빨간색?
- 코드 네이밍을 신경쓴 흔적이 보여 칭찬드립니다.
- 마진, 패딩. flex간 위치 정렬 등 디테일에 좀 더 신경을 써준다면 상당히 완성도 높아질 수 있습니다.
- 커밋컨벤션, release 브랜치 관리 등에 대한 문서화가 부족합니다.
- 팁을 드리자면 다른 사람이 들어왔을때 README만 보고도 바로 코드에 기여할 수 있으려면 어떻게 해야할지 생각해보세요.
- 아직 오류가 있는 부분이 남아있습니다. (지역 선택시 이동시 에러)
UI
- 모바일도 텍스트로 직접 지역을 선택하는 것보다 직접 지도를 클릭하는게 더 낫다는 생각이 듭니다.
- 클릭시 아무 자료가 없는 곳은 반응이 없습니다. 적절한 UI나 팝업을 띄우는건 어떨까요?
- rem과 em을 적절히 잘 사용해보세요. 지금 px단위로만 되어있습니다.
- 애니메이션 속도가 일정하지 않음
일식 등 메뉴선택 애니메이션이 너무 느립니다. 반면 지역 선택은 적절한 속도입니다. 마찬가지로 필터 부분(최신순 , 댓글순, 하트 수 역시 너무 애니메이션이 느립니다)
- 아래와 같은 부분에서 입력 UI를 가리는 부분이 발생합니다
코드 및 협업 관련
README 업데이트 필요
- 프로젝트 실행 방법 및 커밋메시지 작성방법
- 사용기술(react, redux, contextApi, emotion 등..) 명시
- REAMD에 실행방법, package.json script의 역할등을 명시하는게 좋음
- 커밋메시지의 경우 lipstick, recycle등으로 되어있는데 일반적인 프로젝트 형식같지는 않음
- 커밋메시지 접두어가 어떤 내용인지 작성 필요
- 팀원만 적는게 아니라 어떤 역할을 했는지도 구체적으로 명시해주세요.
브랜치 관리
- 클론받았을때 develop브랜치만 존재하여 의아합니다.
- 깃헙 레포에서는 main 브랜치에 PR올리고 합쳐진 로그들이 있지만 클론받았을 때에는 원격 main이 미존재
- Release 관리를 어떻게 하는지도 README에 적어놓기
코드
- component 폴더 안 base 아래 모든 컴포넌트들이 존재하여 폴더를 나눈 의미가 사라진 느낌입니다. 페이지간, 상위하위 component들, 공통요소인 common 폴더등을 활용하여 확장성있게 만들어보시면 좋을것 같아요.
- Pages → createPost → constant는 특정 페이지에 종속되는것 보다는 더 상위 폴더에서 글로벌로 관리가 필요합니다. 제 경우에는 config폴더 아래에 constant로 관리합니다.
- 상위에서 관리하면 ChannelLogo → index→ channelObject 안에 있는 내용들 역시도 constant로 묶어서 관리 할 수 있을듯합니다.
const CITIES = [ { id:1, name:'강남구', englishName:'gangnam', url:'~' } ]
- 미디어 쿼리는 scss에서 mixin 등으로 관리해주는걸 추천드립니다
- 미디어쿼리 사이즈는 클라이언트 요구나, 버그 수정시 자주 바뀌는 경우가 많습니다. scss constant나 mixin으로 관리하면 중복작업을 줄일 수 있습니다.
- 안쓰는 코드들을 제거하고 (특히 console.log)중복된 코드들을 줄여보세요.