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

[고학영] Week2 #19

Conversation

hakyoung12
Copy link
Collaborator

요구사항

기본

  • [기본] PC사이즈만 고려해 주어진 디자인을 구현했나요?

  • [기본] HTML, CSS 파일을 Netlify로 배포했나요? (참고: https://www.codeit.kr/learn/5309)

  • [기본] 랜딩 페이지의 url path는 루트(‘/’)로 설정했나요?

  • [기본] title은 Linkbrary로 설정했나요?

  • [기본] 화면의 너비가 1920px 이상이면 하늘색 배경색은 너비를 꽉 채우도록 채워지고, 내부 요소들의 위치는 고정되고, 여백만 커지도록 했나요?

  • [기본] 화면의 너비가 1920px 보다 작아질 때, “Linkbrary” 로고의 왼쪽 여백 200px “로그인" 버튼의 오른쪽 여백 200px이 유지되고, 화면의 너비가 작아질수록 두 요소간 거리가 가까워지도록 했나요?

  • [기본] Linkbrary 아이콘은 클릭 시 루트 페이지(‘/’)로 이동하나요?

  • [] [기본] 클릭으로 기능이 동작해야 하는 경우, 사용자가 클릭할 수 있는 요소임을 알 수 있도록 cursor: pointer를 설정했나요?

  • [기본] “로그인”은 클릭 시 로그인 페이지(‘/signin’)로 이동하나요?

  • [기본] “링크 추가하기”는 클릭 시 회원가입(‘/signup’)로 이동하나요?

  • [기본] 화면의 너비가 1920px 이상이면 내부에 있는 요소간 동일한 간격을 유지하며 가운데 정렬해야 합니다.

  • [기본] 화면의 너비가 1920px 보다 작아질 때, 최하단에 있는 “codeit-2023”의 왼쪽 여백 104px과 SNS 아이콘들의 오른쪽 여백 104px을 유지하면서 가운데 있는 “Privacy Policy”, “FAQ” 요소와 각각 동일한 간격을 유지하며 가까워져야 합니다.

  • [기본] “Privacy Policy”, “FAQ”는 클릭 시 각각 Privacy 페이지(‘/privacy’), FAQ 페이지(‘/faq’)로 이동하나요?

  • [기본] 페이스북, 트위터, 유튜브, 인스타그램 아이콘은 클릭 시 각각의 홈페이지로 새로운 창이 열리면서 이동 하나요?

  • [랜딩] 아래로 스크롤 해도 “Linkbrary” 로고와 “로그인” 버튼이 있는 상단 네비게이션 바(Global Navigation Bar)영역이 최상단에 고정되게 했나요?

  • 로그인 페이지, 회원가입 페이지 공통] Linkbrary 로고 클릭시 루트 페이지(“/”)로 이동하나요?

  • [로그인 페이지, 회원가입 페이지 공통] 로그인 페이지, 회원가입 페이지 모두 로고 위 상단 여백이 동일한가요?

  • [로그인 페이지, 회원가입 페이지 공통] input 요소에 focus in 일 때, 테두리 색상이 파랑색인가요?

  • [로그인 페이지, 회원가입 페이지 공통] input 요소에 focus out 일 때, 테두리 색상이 회색인가요?

  • [로그인 페이지, 회원가입 페이지 공통] SNS 아이콘들은 클릭시 각각 “https://www.google.com/”, “https://www.kakaocorp.com/page/” 으로 이동하나요?

  • [로그인 페이지] “회원 가입하기”는 클릭시 “/signup” 페이지로 이동하나요?

  • [회원가입 페이지] “로그인 하기”는 클릭시 “/signin” 페이지로 이동하나요?

심화

  • [] [심화] 사용자의 브라우저 font-size가 크고 작아짐에 따라 페이지의 요소간 간격, 요소의 크기, font-size 등 모든 크기와 관련된 값이 크고 작아지도록 설정했나요?

  • [심화] palette에 있는 color값들을 css 변수로 등록하고 사용했나요?

  • [심화] 비밀번호 input 요소 위에 비밀번호를 확인할 수 있는 눈 모양 아이콘을 추가했나요?

주요 변경사항

스크린샷

image

멘토에게

-아직 영역설정과 클래스명 짓기에 미숙합니다

이 부분에 관해서 피드백 주시면 갑사합니다

  • width가 짧아졌을 때 section부분의 배경의 양 옆이 잘려보이는데 이부분은 아직 해결 못했습니다
  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@hakyoung12 hakyoung12 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Feb 24, 2024
@hakyoung12 hakyoung12 requested a review from clianor February 24, 2024 14:41
@clianor
Copy link
Collaborator

clianor commented Feb 27, 2024

과제 제출하실때 프로젝트의 루트에 파일을 풀어서 제출해주세요~

@clianor
Copy link
Collaborator

clianor commented Feb 27, 2024

Q. width가 짧아졌을 때 section부분의 배경의 양 옆이 잘려보이는데 이부분은 아직 해결 못했습니다

A. 해당 내용은 미디어 쿼리에 대해 학습하신 이후에 해결가능하실겁니다 지금은 그렇게 되어도 괜찮습니다~

@clianor
Copy link
Collaborator

clianor commented Feb 27, 2024

index.html 파일에서 지그재그로 되어 있는 부분을 구현하실때 html에서 작성하기를 그렇게 작성해주셨지만 순서가 변경된다거나 할때 대응하기 쉽지 않습니다.
그렇기 때문에 nth-of-type이라는 css 속성과 flex의 flex-direction 혹은 order를 사용하는 방법으로 지그재그로 배치하는 것을 구현할 수 있는데 이렇게 구현하는 것이 좀 더 좋을것 같습니다.

@clianor
Copy link
Collaborator

clianor commented Feb 27, 2024

id를 상당히 많이 이용하고 계시네요 class로 수정해보시면 좋을것 같습니다.

@clianor
Copy link
Collaborator

clianor commented Feb 27, 2024

클래스명을 작성할때는 일관되고 의미론적으로 작성하는 것이 중요합니다.
적절한 접두사는 클래스명 작성에 도움이 되곤합니다.
그러나 중복되는 부분에 대해 한 번 고민을 해보고 signin-form, siginup-form 과 같이 작성하기보다는 auth-form 과 같이 중복을 제거할 수 있는 이름이 더 적절해보입니다.
클래스명을 작성할때 일관성, 의미, 중복 과 관련지어 고민을 해보시면 좋을것 같습니다.

@clianor
Copy link
Collaborator

clianor commented Feb 27, 2024

고생하셨습니다!

@clianor clianor merged commit cdee0bd into codeit-bootcamp-frontend:part1-고학영 Feb 27, 2024
@hakyoung12
Copy link
Collaborator Author

감사합니다!

@hakyoung12 hakyoung12 deleted the part1-고학영-week2 branch March 8, 2024 09:42
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