Skip to content

구현 과제 - 로그인/회원가입 모달 - 의진#30

Open
timetam24 wants to merge 9 commits intomainfrom
11-yooUiJin
Open

구현 과제 - 로그인/회원가입 모달 - 의진#30
timetam24 wants to merge 9 commits intomainfrom
11-yooUiJin

Conversation

@timetam24
Copy link
Copy Markdown

로그인/회원가입 첫 번째 레이아웃을 구현하였습니다.

@AYFG
Copy link
Copy Markdown

AYFG commented Aug 4, 2023

와~로그인 성공 alert 상상도 못했어요😆 헤더에 word-spacing으로 피그마랑 완전 똑같이 구현하려하신 꼼꼼함도 보이네요
h2태그에 아스키코드로 만드셨네요 저는 그냥 " ) " 로 했는데 빨갛게 변하더라고요 덕분에 배웠습니다..
의진님 코드보고 많이 배웠습니다 감사해요!!!👍👍👍👍

@timetam24
Copy link
Copy Markdown
Author

와~로그인 성공 alert 상상도 못했어요😆 헤더에 word-spacing으로 피그마랑 완전 똑같이 구현하려하신 꼼꼼함도 보이네요 h2태그에 아스키코드로 만드셨네요 저는 그냥 " ) " 로 했는데 빨갛게 변하더라고요 덕분에 배웠습니다.. 의진님 코드보고 많이 배웠습니다 감사해요!!!👍👍👍👍

신경 쓴 부분 꼼꼼하게 캐치해주셨네요 감사합니다 재웅님!! 저도 앞으로 많이 보고 배우겠습니다😊 리뷰 다시 한번 감사드려요!!👍

@aicul313
Copy link
Copy Markdown

aicul313 commented Aug 5, 2023

우와 자스까지 넣으시다니 대단해요 😚👍 h2 태그 아스키코드도 그렇고 헤더 h1 태그에 word-spacing으로 글자 간격은 생각도 못했는데 꼼꼼함이 보이는 것 같아요 대단해요 !!! 의진 님 word-spacing 넣으신 것보고 제 코드에도 적용해보아야겠어요 !! 고생하셨습니다 의진 님 🥰

@timetam24
Copy link
Copy Markdown
Author

우와 자스까지 넣으시다니 대단해요 😚👍 h2 태그 아스키코드도 그렇고 헤더 h1 태그에 word-spacing으로 글자 간격은 생각도 못했는데 꼼꼼함이 보이는 것 같아요 대단해요 !!! 의진 님 word-spacing 넣으신 것보고 제 코드에도 적용해보아야겠어요 !! 고생하셨습니다 의진 님 🥰

와아 세세한 리뷰 감사합니다 현지님😊 저도 앞으로 현지님 코드 보고 많이 배울게요!! 같이 좋은 방향으로 성장했으면 좋겠습니다❤️

Copy link
Copy Markdown
Collaborator

@YennieJ YennieJ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

의진님 스프라이트부터 순수 자바스크립트, 시맨틱 태그, 주석까지
깔끔한 코드 잘 봤습니다! 공부가 많이 됐어요 :)
고생하셨습니다!

height: 100vh;
font-family: "SpoqaHanSansNeo-Medium";
font-size: 16px;
color: #767676;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

body에 color자체를 가장 많이 사용한 색으로 하셨네요!!👍

display: block;
width: 28px;
height: 28px;
background-image: url(./images/icon-sprites.png);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

스프라이트를 사용하니 코드가 훨씬 깔끔하네요👍

Copy link
Copy Markdown

@kyokyoYa kyokyoYa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

JS에서 if 문을 어떻게 사용하는 궁금했는데 의진 님 코드를 보면서 해소됐네요!
전반적으로 잘하신 코드라고 생각합니다.
특히 class명을 직관적으로 잘 주셔서 많이 배웠습니다🙌

1. 로그인 성공 후 불일치 안내 메세지가 삭제되지 않는 오류 수정
2. 아이디 미입력 시 기존의 불일치 안내 메세지가 삭제되지 않고 아이디 입력 안내 메세지와 동시 출력되는 오류 수정
아이디 또는 비밀번호 불일치 시 기존의 아이디 미입력 안내 메세지와 focus가 사라지지 않고 불일치 안내 메세지와 동시 출력되는 오류 수정
아이디 미입력 시 기존의 비밀번호 focus가 삭제되지 않고 아이디 미입력 안내 메세지 그리고 아이디 focus와 함께 출력되는 오류 수정
Comment on lines +25 to +29
const checkFocusAndErrorMsg = (input, errorMsg) => {
input.classList.remove("focus");
removeErrorMsg(errorMsg);
};

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

키야 내부함수를 바로 만드시다니 증말 최고애요👍

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ㅎㅎㅎ 감사합니다❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants