-
Notifications
You must be signed in to change notification settings - Fork 373
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[14기 신윤선] step1 돔 조작과 이벤트 핸들링으로 메뉴 관리하기 #273
base: main
Are you sure you want to change the base?
Conversation
README.md에 블랙커피 스터디 14기 함께 공부하는 팀원을 추가했습니다.
에스프레소 메뉴 추가, 수정, 삭제, 총 갯수를 구현했습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good!
} | ||
|
||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
메뉴를 등록한 뒤 생성되는 수정과 삭제 버튼에는 이벤트 위임을 사용하면 좋을것 같아요!
src/js/index.js
Outdated
</button> | ||
</li>` | ||
)); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
변수명을 const li 보다는 li를 생성해준다는 느낌이 들어가면 좋을것 같아요!
🎯 step1 요구사항 - 돔 조작과 이벤트 핸들링으로 메뉴 관리하기
prompt
인터페이스를 활용한다.confirm
인터페이스를 활용한다.😶 개선점
메뉴를 등록 후 수정 버튼을 누르면 이벤트가 실행되지 않는데 pr을 먼저 올린 후에 확인해보려고 합니다.
빈 배열 대신 객체를 넣으면 수정과 삭제 이벤트가 실행이 되는데 새로 추가할 경우에는 실행되지 않아서 이 부분을 개선할 예정입니다.