search

abtinia / ToDo_App

0 stars 0 forks source link

ToDo_App 리뷰 #1

Open Jiiker opened 1 year ago

Jiiker commented 1 year ago

https://github.com/abtinia/ToDo_App/blob/8d72fcc1769795a3be687f6cfa10900457881b30/script.js#L39C5-L51C29 여기는 "LI", "SPAN", "IMG"를 대문자로 작성해주어도 괜찮은 건가요???

listContainer.addEventListener에 click을 이벤트로 지정해 준 이후에 target 설정을 나누어주면 listContainer 내부에서 어디를 클릭하냐에 따라 세부적으로 이벤트를 쪼개줄 수 있다는 것을 깨달았어요..!! 배워갑니다!!

https://github.com/abtinia/ToDo_App/blob/8d72fcc1769795a3be687f6cfa10900457881b30/script.js#L53C7-L60C47 이 부분에서 가상요소 활용하신 부분이 인상깊었어요..!! 저는 아직 제대로 이해를 못한 부분이라 잘 못 쓰겠더라구요...ㅠㅠ

코드가 전반적으로 깔끔하게 정돈된 느낌이라 읽기 편했던 것 같습니다!!~

abtinia commented 1 year ago

2 1 이벤트객체의 target.tagName을 통해 반환되는 값이 'LI', 'SPAN', 'IMG' 처럼 대문자로 나와서 조건식도 대문자로서의 태그값을 검사하게 했습니다!

저도 사실 가상요소 부분은 코드를 읽고 사용법을 이해하는 정도일 뿐이라.. 제가 직접 생각해서 저런 코드를 작성하진 못하겠더라구요... 사고회로가 단순해서 그런지 참 스스로가 안타깝습니다.. ㅠㅠ

리뷰 감사합니다!