YongHoonJJo commented 4 years ago

1 Mission001 / TodoList CRUD

[x] todo list에 todoItem을 키보드로 입력하여 추가
[x] todo list의 체크박스를 클릭하여 complete 상태로 변경. (li tag 에 completed class 추가)
[x] todo list의 x버튼을 이용해서 해당 엘리먼트를 삭제
[x] todo list를 더블클릭했을 때 input 모드로 변경. (li tag 에 editing class 추가) 단, 이때 수정을 완료하지 않은 상태에서 esc키를 누르면 수정되지 않은 채로 다시 view 모드로 복귀
[x] todo list의 item갯수를 count한 갯수를 리스트의 하단에 보여주기
[x] todo list의 상태값을 확인하여, 해야할 일과, 완료한 일을 클릭하면 해당 상태의 아이템만 보여주기
예전에 제가 진행했던 미션을 참고하면서 진행했는데, 구조를 어떤식으로 잡아야 할지, 설계를 하는 것이 역시나 제일 어려웠던 것 같습니다.

YongHoonJJo commented 4 years ago

@EastjunDev

리뷰주셨던 내용들에 대해 모두 반영하였습니다. 꼼꼼히 살펴봐주셔서 감사합니다!!

eastjun-dev commented 4 years ago

@YongHoonJJo 용훈님 의견 남겨드린 부분 모두 정확히 반영해주시고, 제 코드에도 좋은 피드백 많이 남겨주셔서 너무 감사드립니다 :) 이번미션은 approve진행할게요 😄

StellaKim1230 commented 4 years ago

@YongHoonJJo 컴포넌트 설계에 대해서 같이 고민을 나누면 더 좋을거 같아요 ㅎㅎ (저도 항상 고민중이라...) 이번미션 approve 하겠습니다^^ 👍

YongHoonJJo commented 4 years ago

@amorfati0310

e.target 캐싱 처리
document.querySelectorAll('ul.filters li a') 에서 document.querySelectorAll('.selected') 로 변경
data-actinon 과 data-id 분리
++ 연산자 대신, +=1 로 사용.

피드백 주신 내용 모두 반영하였습니다. :)

eastjun-dev / frontend