Conversation
kimsiyeon0223
requested review from
anys34,
kimsiyeon0223,
seonghoon07 and
winternuary
|
우선 너무너무 수고하셨습니다! 시간도 잘 지켜주셔서 감사하네용😊 코드를 살짝 봤는데 제가 저번에 말했던 rgb 지양하고 색상코드 사용한 점! 이 가장 먼저 보였어요. 깃허브가 젤 어려운 거 정말 공감해요. 저도 1학년 때 다 개발한 브랜치를 날려먹어서 울면서 수고 많으셨고 이번에도 남긴 리뷰 확인 해 보시고 리팩토링하는 과정 거치시면 좋을 것 같아요 |
|
|
||
| input() | ||
|
|
||
| result = person1===person2 ? '같다' : ((person1>person2) ? '첫번째가 더 크다' : '두번째가 더 크다' ) //삼항 연산자, Strict Equal 연산자 |
There was a problem hiding this comment.
주석 남기며 공부하는 방법 너무 좋은 것 같습니다!!
| border-radius: 3px; | ||
| } | ||
|
|
||
| #textcontainer{ |
There was a problem hiding this comment.
다 너무 좋지만 id나 class 네이밍 컨벤션 같은 것도 알고계시면 좋을 것 같아요!
https://velog.io/@yeonsubaek/Naming-Convention네이밍-컨벤션-종류와-예시
| li.appendChild(text) | ||
| li.appendChild(delbtn) | ||
|
|
||
| const todo = document.querySelector('#textcontainer') |
There was a problem hiding this comment.
querySelector 보다는 getElementById 나 getElementsByClassName 를 사용하는 방법이 있습니다.
더 빠르고 더 잘 지원이 된답니다!!
https://velog.io/@chloeee/getElementById-그리고-querySelector-차이점
| btn.addEventListener('click', () => { | ||
| if (swt) { | ||
| li.style.textDecoration = "line-through" | ||
| li.style.color = "#a1a1a1" | ||
| btn.style.backgroundColor = "#e37e8e" | ||
| swt = false | ||
| } // 체크 안되어 있을 시 버튼 누르면 밑줄 긋기 | ||
| else { | ||
| li.style.textDecoration = "none" | ||
| li.style.color = "black" | ||
| btn.style.backgroundColor = "white" | ||
| swt = true | ||
| } // 체크 되어 있을 시 버튼 누르면 밑줄 삭제 | ||
| }) // 버튼 한번 클릭 시 텍스트에 줄 긋기 |
There was a problem hiding this comment.
만약 swt의 true false 값에 따라 if else를 나눈 거라면, 삼항연산자를 사용하는 것도 추천드립니다!!
li.style.textDecoration = swt ? "line-through" : "none";요런 식으로요...
| @@ -0,0 +1,23 @@ | |||
| <!DOCTYPE html> | |||
| <html lang="en"> | |||
There was a problem hiding this comment.
문서의 주 언어가 한국어로 작성된 경우, lang 속성을 ko로 변경하는 것이 좋습니다!!
그렇지 않으면 새로고침 할 때 마다 계속해서 언어를 물어보게 될 수 있어용
|
피드백 주신 내용을 반영하여 수정 완료했습니다. |
2 participants
흥미 있었던 점 : JS로 직접 프로젝트를 만들어 보니 웹 개발에 대해 더 흥미가 생겼습니다. 기초적인 문법이 아닌 어디에 적용 해야할지 명확하게
알 수 있어서 좋았습니다. 레퍼런스에 정말 이해하기 쉽도록 적혀 있었기에 내용을 읽으면서 막히는 부분이 없었습니다. 또한 클론코딩이 아닌 직접 웹사이트를 구현하는게 쉽지만은 않았지만 그래도 클론코딩보다 더 재밌다고 느꼈습니다.
어려웠던 점 : 뭐가 문제였는지 pr이 안되어 정말 이악물고 찾아봤지만 문제를 해결하지 못했습니다.. 그래서 결국 선배에게 도움을 요청하여 해결
하였습니다. github를 다루는 것이 개발보다 더 어려운거 같습니다. 그리고 선배들이 더욱 대단하다고 느꼈습니다.. 존경합니다.
궁금한 점 : 저번과 똑같이 하였는데 이번에 pr이 안된 정확한 이유를 모르겠습니다. 깃허브에 대해 더욱 공부 해야겠다는 생각이 들었습니다.