Skip to content
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

[1단계 - todo list] 크로플(이동환) 미션 제출합니다. #39

Open
wants to merge 1 commit into
base: perenok
Choose a base branch
from

Conversation

perenok
Copy link

@perenok perenok commented Mar 27, 2021

js 너무 어렵습니다 흑흑ㅠㅠㅠ
분명 맞게 했다고 생각하고 돌려보면 대답없는 너란 이벤트들ㅠㅠㅠ
당장 미션을 진행하기 위해 필요한 기능들을 인터넷을 통해 찾아가며 적용은 시켰지만 이게 맞는건지에 대한 확신이 없네요ㅎㅎㅎ
그래도 어찌어찌 만들어서 내가 원하는 이벤트가 원하는 때에 나타나면 이거대로 짜릿하더라고요ㅋㅋㅋ
프론트 분들이 새삼 존경스러워지네요 전 역시 백엔드가...ㅎㅎㅎ

궁금한 점

  1. 요구사항에 toggle을 체크할 시 checked 속성을 넣는 게 있었는데 여기서 checked 속성을 처음에 넣고 다시 클릭하면 이 checked = false로 바꾸는 법이 setAttribute 말고는 없는건가요? 일단 지금은 다시 체크 해제 하면 checked 속성을 없애는 식으로 하는데 setAttribute할 때 속석의 value값을 true로 넣어주다 보니 false로 바뀌는 식으로 가면 깔끔할 거 같아서요ㅎㅎ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant