-
Notifications
You must be signed in to change notification settings - Fork 15
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
Basic 엄성민 sprint3 #28
The head ref may contain hidden characters: "basic-\uC5C4\uC131\uBBFC-sprint3"
Basic 엄성민 sprint3 #28
Conversation
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.
성민님 고생하셨어요!
사실 js 들어가면서부터 �익숙하지 않아 어려울수도 있는데, 노력의 흔적이 많이 보입니다!
전체적으로 잘 작성해주신 것 같고, 다만 중복된 코드들은 한번만 쓸수 있게 재사용할수 있게 해주는게 좋습니다. �그리고 최상단에 project라는 디렉토리안에 넣을 필요없이 그냥 파일들을 놔둬도 될것 같아요 최상단에는 project만 있으니까 불필요한것 같습니다!
다음에는 배포까지 해주시면 더 자세히 리뷰해드릴 수 있을것 같아요~
정말 수고 많으셨고, 발전하시는게 눈에 보입니다!
궁금하신것 채널이나 PR에서 언제든 물어봐주세요~
@@ -0,0 +1,175 @@ | |||
const USER_DATA = [ |
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.
이렇게 공통된 애들은 따로 constant.js 같이 빼놓는게 더 좋을듯 해요!
.error_msg p{ | ||
margin: 0; | ||
padding-bottom: 40px; | ||
|
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.
요렇게 빈칸같은 것들 지난번에 말씀드린 prettier 같은걸 적용하면 한번에 다 없어져요~
|
||
inputbox.forEach(input=>{ | ||
input.addEventListener('input',()=>{ | ||
console.log(input.value); |
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.
PR 생성시에는 console.log 있는지 확인후 지운다음에 올려주시면 좋을것 같아요~
input.addEventListener('input',()=>{ | ||
console.log(input.value); | ||
if(input.id === "section1_emailbox"){ | ||
if(email_pattern.test(input.value)){ |
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.
요 if 문을 아래와같이 더 간결하게 바꿀수 있을것 같아요!
email_check = email_pattern.test(input.value);
} | ||
|
||
else if(input.id === "section1_pwbox"){ | ||
if(pw_pattern.test(input.value)){ |
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.
여기도 같습니다! pw_check = pw_pattern.test(input.value);
|
||
|
||
//입력 값이 올바른지 체크 | ||
inputbox.forEach(input=>{ |
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.
중복되는 로직들은 따로 파일을 빼서 각 페이지마다 넣어줘도 좋을것 같아요!
예를 들면, input.js 를 만들고, login.html, singup.html에 스크립트를 넣어주기만 하는거죠
요구사항
기본
공통
로그인, 회원가입 페이지 공통
로그인 페이지
만약 입력한 이메일이 데이터베이스(USER_DATA)에 없거나, 이메일은 일치하지만 비밀번호가 틀린 경우, '비밀번호가 일치하지 않습니다.'라는 메시지를 alert로 표시합니다
만약 입력한 이메일이 데이터베이스에 존재하고, 비밀번호도 일치할 경우, “/items”로 이동합니다.
회원가입
입력한 이메일이 이미 데이터베이스(USER_DATA)에 존재하는 경우, '사용 중인 이메일입니다'라는 메시지를 alert로 표시합니다.
입력한 이메일이 데이터베이스(USER_DATA)에 없는 경우, 회원가입이 성공적으로 처리되었으므로 로그인 페이지(”/login”)로 이동합니다.
심화
공통
랜딩 페이지
PC: 1200px 이상
Tablet: 744px 이상 ~ 1199px 이하
Mobile: 375px 이상 ~ 743px 이하
375px 미만 사이즈의 디자인은 고려하지 않습니다
로그인, 회원가입 페이지 공통
주요 변경사항
-https://fascinating-alpaca-7788c5.netlify.app/
-틈틈히 수정 할 예정입니다.
스크린샷
멘토에게
-아직 구현중입니다.