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

[Mission2] ID, PW 입력 시 로그인 버튼 활성화 기능 #2

Open
saengmotmi opened this issue Jul 14, 2021 · 0 comments · Fixed by #452
Open

[Mission2] ID, PW 입력 시 로그인 버튼 활성화 기능 #2

saengmotmi opened this issue Jul 14, 2021 · 0 comments · Fixed by #452

Comments

@saengmotmi
Copy link
Contributor

saengmotmi commented Jul 14, 2021

✔️ 해당 미션과 관련하여 코드 리뷰에서 중점적으로 체크하는 사항입니다.


변수 선언 키워드, 변수 선언 위치

변수를 선언하는 키워드는 세가지, var, let, const 가 있습니다. 변수 선언 시 아무 이유 없이 섞어서 쓰는게 아니라 항상 분명한 이유를 갖고 적절한 키워드를 선택해 사용해주세요.

변수를 함수 밖에서 선언하는지, 함수 안에서 선언하는지에 따라서 기능이 동작하고 안 하고의 차이를 발생시킵니다. 변수를 어느 위치에서 선언하면 좋을지, 이 변수가 다른 함수에서도 사용되는지 등의 사항들을 고려해주세요.

📍 관련 PR & Review 보기

📎 참고자료


addEventListener

addEventListener 는 DOM에서 내가 원하는 요소에 이벤트를 실행시키기 위해 반드시 사용되는 메소드입니다. 어떤 요소에 이벤트를 부여할지 충분히 고민해보세요 :)

📍 관련 PR & Review 보기

📎 참고자료


삼항 조건 연산자

삼항 연산자는 실제 개발을 하며 자주 사용되는 if 명령문의 단축 형태로 쓰이는 문법입니다. 로그인 버튼 활성화 기능을 구현하며 삼항 연산자를 반드시 적용해보세요.

📎 참고자료

@SK0414 SK0414 linked a pull request Sep 9, 2021 that will close this issue
3 tasks
@SK0414 SK0414 removed a link to a pull request Sep 9, 2021
3 tasks
@KH2Wone KH2Wone linked a pull request Dec 8, 2021 that will close this issue
10 tasks
@KH2Wone KH2Wone removed a link to a pull request Dec 11, 2021
10 tasks
@seoltang seoltang linked a pull request Jan 5, 2022 that will close this issue
5 tasks
@hyunkark hyunkark linked a pull request Jan 6, 2022 that will close this issue
5 tasks
@wkdwhdghks wkdwhdghks linked a pull request Mar 11, 2022 that will close this issue
5 tasks
@keemhyesu keemhyesu reopened this May 5, 2022
@DevSeulgi DevSeulgi linked a pull request May 6, 2022 that will close this issue
5 tasks
@AliceNam93 AliceNam93 linked a pull request Oct 28, 2022 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants