[전정민]Sprint2

[miimmm]

요구사항

기본

• “판다마켓" 클릭 시 루트 페이지(“/”)로 이동합니다.
• 로그인 페이지, 회원가입 페이지 모두 로고 위 상단 여백이 동일합니다.
• input 요소에 focus in 일 때, 테두리 색상은 #3692FF입니다.
• input 요소에 focus out 일 때, 테두리는 없습니다.
• SNS 아이콘들은 클릭시 각각 아래 페이지로 이동합니다.

• “로그인”버튼 클릭 시 (“/login”) 페이지로 이동합니다.
• “회원 가입”버튼 클릭 시 “/signup” 페이지로 이동합니다.

심화

• 비밀번호 input 요소 오른쪽에 눈모양 아이콘을 추가합니다.
(추후 클릭으로 비밀번호를 보거나 가릴 수 있도록 기능을 만들어 갈 예정입니다.)
• palette에 있는 color값들을 css 변수로 등록하고 사용합니다.

주요 변경사항

• 회원가입, 로그인 페이지 구현
• input focus시 스타일 적용
• input 값 입력시 버튼 활성화

스크린샷

멘토에게

• git 이렇게 올라가면 괜찮은걸까요?
• 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

[baeggmin]

고생많으셨습니다! PR 잘 올려주셨어요 ㅎㅎ

근데 체크하신 구현 사항과 실제 코드가 상이한것같아요.
코드에서는 login 페이지가 Empty file 로 올라왔는데 이게 맞나요??
다시 한 번 확인 부탁드릴게요~

[baeggmin]

css 변수 설정 잘 해주셨네요! 👍

[baeggmin]

혹시 맥북으로 하는 첫 프로젝트 셋팅이신가요?
.DS_Store 파일은 보안상 올리지 않는 편이 좋습니다.
아래 블로그 글에 해결 방법이 잘 정리되어 있으니 참고하셔서 제거해주세요!

https://velog.io/@junr_65535/.DSstore%EA%B0%80-%EB%AD%90%EC%97%AC-%EB%82%9C-%EC%9D%B4%EB%9F%B0%EA%B1%B0-%EC%93%B4%EC%A0%81-%EC%97%86%EB%8A%94%EB%94%94

[baeggmin]

classname 지을 때 단어가 너무 길면 약어를 쓰기도 하지만, 지금 같은 경우에는 그렇게 길지 않은 단어라 굳이 줄이지 않아도 좋을 것 같아요. 개발자마다 약어를 정하는 방법이 가지각색이라 혼동의 여지가 있을 수 있어서요!

[baeggmin]

여기 Rvrs 는 무엇을 의미하는건가요? 그리고 중간에 하이픈이 빠진 것 같습니다. classname 지으실 땐 유지보수성을 위해 가급적 하나의 스타일로 통일해주시는게 좋습니다~!

[baeggmin]

img 태그 쓰실 땐 가급적 alt 속성 값을 채워주세요~!

[baeggmin]

section, main, header, footer 등 시맨틱 태그를 잘 사용해주셨네요!

[baeggmin]

.ds_stroe 제거하시면 머지하려고 했는데 일단 지금 머지할게요!