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

[신승헌]sprint 2 #117

Conversation

AdamSeungheonShin
Copy link
Collaborator

요구사항

기본 요구사항

  • Github에 PR(Pull Request)을 만들어서 미션을 제출합니다.
  • Netlify에 파일 배포가 아닌 포크한 Github 레포지토리로 연결합니다.
  • 피그마 디자인에 맞게 페이지를 만들어 주세요.
  • React와 같은 UI 라이브러리를 사용하지 않고 진행합니다.

기본

###로그인 페이지, 회원가입 페이지 공통

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

###로그인 페이지

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

###회원가입 페이지

  • “로그인”버튼 클릭 시 “/login” 페이지로 이동합니다.

심화

  • palette에 있는 color값들을 css 변수로 등록하고 사용해 주세요.
  • 비밀번호 input 요소 위에 비밀번호를 확인할 수 있는 아이콘을 추가해 주세요.

주요 변경사항

스크린샷

mission2_회원가입
mission_로그인

멘토에게

  • Figma 구성을 토대로 div 태그들을 구현 하려고 했는데, div 태그 안에 또 다른 div태그(태그안에태그안에태그)가 너무 많이 생기는 것 같습니다..
  • 자손(자식) div 태그의 개수(단계)가 많아져도 상관 없을까요..?

@AdamSeungheonShin AdamSeungheonShin added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Mar 22, 2024
@AdamSeungheonShin AdamSeungheonShin changed the title Part1 신승헌 sprint2 [신승헌] Sprint 2 Mar 22, 2024
@AdamSeungheonShin AdamSeungheonShin changed the title [신승헌] Sprint 2 [신승헌]sprint 2 Mar 22, 2024
Copy link
Collaborator

@jyh0521 jyh0521 left a comment

Choose a reason for hiding this comment

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

1주차 리뷰에 남겨드렸던 내용들 반영이 아직안되서 1주차것도 함께 봐주시면 감사하겠습니다! 고생하셨습니다 :)

}

html {
margin: 0;
Copy link
Collaborator

Choose a reason for hiding this comment

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

탭이 스페이스바 4칸 크기인것 같은데, 웹 개발할때는 보통 2칸을 사용합니다.

header {
border-bottom: 1px;
background-color: #ffffff;
position: fixed;
Copy link
Collaborator

Choose a reason for hiding this comment

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

fixed와 비슷한 sticky라는 값도 있는데, 둘의 차이를 한번 알아보시면 좋을것 같아요!

@jyh0521
Copy link
Collaborator

jyh0521 commented Mar 24, 2024

  • Figma 구성을 토대로 div 태그들을 구현 하려고 했는데, div 태그 안에 또 다른 div태그(태그안에태그안에태그)가 너무 많이 생기는 것 같습니다..
    => div가 어느정도 생기는것은 정상입니다. ㅎㅎ 다만, 시멘틱 태그 개념도 활용해보시면 좋을것 같고, div가 너무 많이 생기는것 같다면 css 선택자 같은것들을 활용해서 줄여보려고 고민해보시는 것도 도움될 것 같아요!

  • 자손(자식) div 태그의 개수(단계)가 많아져도 상관 없을까요..?
    => 엄청나게 많아지면 가독성도 떨어지고 좋진 않겠지만, 2 ~ 4뎁스 정도는 상황에 따라서 충분히 생길만 합니다!

@jyh0521 jyh0521 merged commit fb7d22a into codeit-bootcamp-frontend:part1-신승헌 Mar 24, 2024
@AdamSeungheonShin AdamSeungheonShin deleted the part1-신승헌-sprint2 branch April 23, 2024 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants