-
Notifications
You must be signed in to change notification settings - Fork 44
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 #57
The head ref may contain hidden characters: "part1-\uC870\uC218\uC5F0"
[조수연] Week2 #57
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
첫번째 미션 고생하셨습니다~!
pr 도 기한 맞춰 제출하시고 작업물의 양도 좋았습니다.
html 구조를 잡으실 때는 피그마를 많이 참고하신 것 같아요.
그러다보니 불필요하게 중첩된 태그나 적용된 스타일 속성들이 보였는데요.
앞으로 미션 해나가면서 개선해나갈 수 있는 부분들이니 리뷰 확인해보시고 다음주 미션 때 반영해보시면 좋을 것 같습니다.
@@ -0,0 +1,118 @@ | |||
<!DOCTYPE html> | |||
<html lang="en"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lang 속성값으로 en 을 지정해주셨네요.
lang 은 사용할 언어를 정의하는 것이고 한국어로 디자인된 사이트를 작업하고 있으니
한국어를 나타내는 값으로 수정해야 합니다.
찾아보시고 수정해주세요!
<body> | ||
<header> | ||
<div class="nav"> | ||
<a href="index.html"><img class="logo" src="logo.svg" alt="로고"></a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
html 구조를 파악하기 쉽도록 들여쓰기를 신경써주세요.
<div class="nav">
<a href="index.html"></a>
</div>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alt 속성값도 잘 작성해주셨네요~
<link rel="stylesheet" href="index.css"> | ||
</head> | ||
<body> | ||
<header> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
시맨틱 태그를 잘 적용해주셨네요~
</div> | ||
</header> | ||
|
||
<section class="contents01"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
class 네이밍을 좀 더 의미있게 하면 좋을 것 같아요.
contents{0n} 만 봐서는 어떤 부분을 의미하는지 알기 어려워서요.
mdn 에도 나와있듯 요소의 의미와 목적을 생각해서 수정해보시면 좋을 것 같습니다.
요소의 의미와 목적을 설명하는 명칭을 사용하는 것이 좋습니다.
제가 지어보자면 사이트를 소개하면서도 가장 메인이 되는 문구가 있어서 main-description
정도로 생각해봅니다.
https://developer.mozilla.org/ko/docs/Web/HTML/Global_attributes/class
|
||
<section class="contents01"> | ||
<div class="title01"> | ||
<span class="txt1">세상의 모든 정보</span><span class="txt2">를<br>쉽게 저장하고 관리해 보세요</span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
color 속성값의 상속을 이용하면 span 태그를 두개를 사용하지 않고도 디자인 구현이 가능할 것 같아요.
이건 여유가 되면 고민해보세요!
|
||
.title01 .txt2 { | ||
color: #000; | ||
font-family: 'pretendard-Regular'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
텍스트마다 font-family 를 설정해주셨습니다.
일일히 지정하지 않도록 하는 방법이 있으니 찾아서 수정해보세요.
text-decoration: none; | ||
} | ||
|
||
.example01 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
예시 이미지군요. 이미지임을 알 수 있도록 home-img 등으로 클래스 명을 수정하면 좋을 것 같습니다.
또한 구현된 걸 보니 가로세로 비율이 깨져보입니다. 확인 후 수정해 주세요.
|
||
.frame04 .txt11 { | ||
display: flex; | ||
width: 181.111px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
소숫점 값은 브라우저마다 렌더링하는 방식의 차이가 있는데요.
스펙 상 width 값으로는 정수만 지원이 되니 정수로 수정해주세요.
|
||
.frame03 { | ||
display: flex; | ||
flex-direction: column; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
들여쓰기 신경써주세요~
</div> | ||
<div class="input"> | ||
<div class="info"> | ||
<label>이메일</label> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
여유가 된다면 label 태그의 for 속성을 찾아 어떤 역할을 하는지 확인해보고 적용해보세요!
요구사항
기본
심화
주요 변경사항
스크린샷
https://linkbrary-week2-mission.netlify.app/
<메인화면>
<로그인 창>
<회원가입 창>
멘토에게