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 #58

Conversation

bomin0830
Copy link
Collaborator

@bomin0830 bomin0830 commented Feb 25, 2024

요구사항

기본

  • [기본] 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 요소 위에 비밀번호를 확인할 수 있는 눈 모양 아이콘을 추가했나요?

주요 변경사항

스크린샷

  • index.html
    Screenshot 2024-02-26 at 15 07 06
  • signin.html
    image
  • signup.html
    image

멘토에게

  • 시간 조절을 잘못해서 심화까지는 진행을 못했습니다. 월요일까지 도전해보겠습니다.
  • 요구사항 5,6번 : url path를 따로 페이지경로(signup.html 혹은 signin.html)이 아닌 파일명만 적어도 가능하게 하는 방법을 모르겠습니다.

@bomin0830 bomin0830 requested a review from choinashil February 25, 2024 14:44
@bomin0830 bomin0830 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Feb 25, 2024
@bomin0830 bomin0830 changed the title [김보민] [김보민] Week2 Feb 25, 2024
Copy link
Collaborator

@choinashil choinashil left a comment

Choose a reason for hiding this comment

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

처음 하는 과제라 어려우셨을텐데 끝까지 마무리하시느라고 수고 많으셨습니다!
그래도 기본적인 html, css 에 대해서 이해를 잘 하고 계신 것 같아요
디테일한 부분만 앞으로 배워나가면 더 좋아질 것 같습니다 👍🏽

코멘트 달린 부분들 꼼꼼하게 확인하시고 다음 미션에 반영 부탁드립니다!

table {
border-collapse: collapse;
border-spacing: 0;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

각 파일마다 마지막에 빈 줄을 하나씩 넣어주는 것을 권장해요.

(찾아보기 키워드: EOL, end of line, 파일 마지막 개행)

@@ -0,0 +1,164 @@
@import "reset.css";
Copy link
Collaborator

Choose a reason for hiding this comment

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

@import 잘 활용하셨네요 👍🏽
reset css 파일 분리해서 이렇게 참조하는 것 좋습니다!

Comment on lines +6 to +7


Copy link
Collaborator

Choose a reason for hiding this comment

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

클래스 사이에 빈 줄은 1줄만 띄어주세요

}


html{
Copy link
Collaborator

Choose a reason for hiding this comment

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

태그, 클래스와 중괄호 사이에 1칸 띄어쓰기 해주세요!

Suggested change
html{
html {



html{
height:100%;
Copy link
Collaborator

Choose a reason for hiding this comment

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

속성의 키와 값 사이에도 1칸 띄어쓰기 해주세요!
전체적으로 확인 및 수정해주시고 앞으로의 과제에서도 요런 부분은 실수 없도록 꼼꼼히 확인 부탁드립니다!

Suggested change
height:100%;
height: 100%;

</header>
<main>
<section class="main">
<h1 class="main-title">
Copy link
Collaborator

Choose a reason for hiding this comment

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

h1 은 해당 페이지에서 가장 중요한 제목을 담습니다
이 페이지에서 제목 역할을 하는 부분은 어디일까요??

</div>
<h2>
회원이 아니신가요?
<a href="./signup.html">회원가입하기</a>
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기는 button 태그 대신 a 태그 잘 사용해주셨네요! 💯

요구사항 5,6번 : url path를 따로 페이지경로(signup.html 혹은 signin.html)이 아닌 파일명만 적어도 가능하게 하는 방법을 모르겠습니다

질문주셨는데, 확장자를 포함하지 않고 파일명만 활용하려면 추가적인 설정이 필요하기 때문에 우선은 지금처럼 적용하시면 됩니다 :)

</div>
<div class="input-form">
<div class="email-input">
<h3>이메일</h3>
Copy link
Collaborator

Choose a reason for hiding this comment

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

input 의 이름은 label 태그를 사용해주세요.

label 과 input 을 연결하기 위해 쓰이는 id, for 등의 속성도 공부해서 같이 추가해보세요!

<div class="input-form">
<div class="email-input">
<h3>이메일</h3>
<form>
Copy link
Collaborator

Choose a reason for hiding this comment

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

form 태그는 여러 인풋들을 묶는 데 사용합니다

지금은 input 하나하나를 form 으로 각각 감싸주셨는데
하나의 form 태그 안에 이메일과 비밀번호 인풋 두개가 다 포함되도록 구조를 개선해주세요!

폼 쪽 구조 잡는 부분이 어렵긴 한데 중요한 부분이라서
여러 자료들 검색해보시면서 따라해보시면 좋을 것 같아요

}

input:focus {
outline: 1px #6D6AFE solid;
Copy link
Collaborator

Choose a reason for hiding this comment

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

border 로 처리하시면 됩니다!

@choinashil choinashil merged commit 0fb9bca into codeit-bootcamp-frontend:part1-김보민 Feb 27, 2024
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