Conversation
yona/week1/mission1/index.html
Outdated
| @@ -0,0 +1,32 @@ | |||
| <!DOCTYPE html> | |||
| <html lang="en"> | |||
There was a problem hiding this comment.
페이지 내용이 한국어이기 때문에 lang="ko"로 설정해주는 것이 좋습니다 !
이렇게 설정하면 검색엔진이 페이지 언어를 정확하게 인식해서 SEO(검색 노출)에 도움이 됩니다
yona/week1/mission1/style.css
Outdated
| text-align: left; | ||
| } | ||
|
|
||
| .render-container__title{ |
There was a problem hiding this comment.
.render-container__title 클래스가 중복으로 정의되어 있는 것 같습니다 !
동일한 스타일이 두 번 작성되어 있어 하나는 제거해주셔도 좋을 것 같아요.
yona/week1/mission1/src/script.ts
Outdated
|
|
||
|
|
||
|
|
||
| /*<ul id="todo-list" class="render-container__list"> |
There was a problem hiding this comment.
script.ts에 HTML 코드가 주석으로 남아있는 것 같아요 ! 사용하지 않는 코드라면 지워주시면 코드가 더 깔끔해질 것 같습니다 🙂
yona/week1/mission1/index.html
Outdated
| <!--JS파일 업로드--> | ||
| <script type="module" src="./dist/script.js" defer ></script> | ||
|
|
||
| <title>Document</title> |
There was a problem hiding this comment.
현재 기본값으로 되어 있는데 의미있는 제목으로 바꿔봐도 좋을 것 같아요 !
There was a problem hiding this comment.
현재 0주차 미션과 1주차 미션이 동일한 PR에 묶여서 제출되었는데 다음번부터는 아래 중 한 방법으로 분리해주시면 좋을 것 같습니다 😄
브랜치 분리: week0/yona-m1, week1/yona-m1 처럼 주차별 브랜치를 따로 만들어 PR 분리
커밋 메시지 구분: 하나의 브랜치에서 작업하더라도 [0주차] 미션1 완료, [1주차] 미션1 완료 처럼 커밋 메시지로 구분
yewon20804
left a comment
yewon20804
left a comment
There was a problem hiding this comment.
BEM 네이밍과 함수 분리가 잘 되어 있습니다 !
미션하시느라 수고 많으셨고 리뷰 확인 부탁드립니다 🙂
2 participants
투두리스트 구현하기