Kdt0 nam hyeon jun#58
Conversation
✅ Deploy Preview for velvety-puppy-c1029b ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
운동일지까지 기록할 수 있네요 ㅋㅋ |
|
css 적인 부분은 조금 아쉽긴 하지만, 기능적인 부분에서 엄청 노력하신 것이 보입니다. |
Eojoonhyuk
left a comment
Eojoonhyuk
left a comment
There was a problem hiding this comment.
아이디어가 재밌네요ㅎㅎ CSS까지 추가되면 더 좋은 퀄리티가 될거같습니다!
|
MVC 구조로 작업하셔서 파일을 각 페이지별로 나눈 뒤 라우터를 둔다면 좀 더 직관적인 폴더 구조를 가질 수 있을 것 같아요! |
iamidlek
left a comment
iamidlek
left a comment
There was a problem hiding this comment.
필수 요소와 기능은 잘 현해 주신 것 같습니다.
css 부분이 조금 아쉬운 것 같습니다.
일부 일반적으로 권장되는 스타일을 코멘트 하였고
중복되는 부분은 생략하였습니다.
고생하셨습니다.
| <ul> | ||
| <li>세션 번호</li> | ||
| <li>날짜</li> | ||
| <li>부위</li> | ||
| <li>내용</li> | ||
| <li>중량</li> | ||
| <li>횟수</li> | ||
| <li>세트</li> | ||
| <li>비고</li> | ||
| </ul> |
| userPtSession, | ||
| userTrainer | ||
| ); | ||
| userBox.append(userInfoFrame); |
There was a problem hiding this comment.
반복문 안에서 append나 innerHTML을 하기 보단
변수에 모든 내용을 가지고 한번에 적용하는 것이 좋을 것 같습니다.
| ) { | ||
| const db = getDatabase(); | ||
| try { | ||
| console.log(sessionId, date); |
There was a problem hiding this comment.
확인이 완료된 부분은 console.log를 제거하면 좋을 것 같습니다.
…VASCRIPT_PICTURE into KDT0_NamHyeonJun
6 participants
🏋️♂️ 헬스장 회원 관리 서비스
직원들의 사진을 관리할 수 있는 사진 관리자 서비스를 만들어 보세요.
배포 주소
https://nhj-js-homework.firebaseapp.com/index.html
[필수 요구사항]
[선택 요구사항]
유저 플로우
화면 예시
메인 페이지
등록 페이지
프로필 페이지
수정 페이지
느낀 점
필수 기능을 먼저 빠르게 완성시켜 놓은 다음 이후 기능 구현을 진행했어야 했는데, 처음부터 기획을 너무 크게 잡고 진행해 속도도 나지 않았고, 목표로 한 기능들도 전부 구현하지 못해 스크립트 부분만 빠르게 진행하였고, css 스타일링 부분은 많이 진행하지 못했던 점이 아쉬웠습니다.
현재 필수 기능 중 데스크탑-모바일 반응형 개발 부분은 미구현 상태입니다.
추후 리팩토링 진행하면서 구현 예정입니다.