-
Notifications
You must be signed in to change notification settings - Fork 43
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
[남기연] sprint1 #37
The head ref may contain hidden characters: "Basic-\uB0A8\uAE30\uC5F0-sprint1"
[남기연] sprint1 #37
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.
미션 하느라 수고 많으셨어요~👏🏻
justify-content: center; | ||
align-items: center; | ||
width: 100%; | ||
font-family: "Pretendard"; |
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.
현재 Pretendard 폰트 적용되지 않는 상태에요.
Pretendard는 브라우저에서 기본 제공하는 폰트가 아니라 font파일을 가져와야 해요.
참고해서 폰트 파일을 가져와 주세요.
<a | ||
href="http://twitter.com" | ||
target="_blank" | ||
rel="noopener noreferrer" |
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.
noopener noreferrer 설정 좋아요~👍
><img | ||
id="logo_Img" | ||
src="/assets/imgs/logoImg.png" | ||
alt="로고 이미지" |
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 설정 좋아요~👍
</header> | ||
<main> | ||
<section> | ||
<img src="/assets/imgs/Img_home_01.png" alt="Img_home_01" /> |
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 들어갈 단어에 고민이 생길 때 참고하면 좋아요.
<div class="emp"></div> | ||
<div class="headerNav"> | ||
<a href="/" | ||
><img |
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.
a 태그 닫히는 >위치가 어색한데
<a href="/">
<img
src="~"
alt="~"
...
/>
<a />
같이 끊어주는게 일반적일 것 같아요.
<div class="headerNav"> | ||
<a href="/" | ||
><img | ||
id="logo_Img" |
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를 권장해요.
- id 는 보통 개발작업시 요소 선택을 위한 방편으로 많이 사용해요.
- id 선택자는 class보다 우선순위가 높아 id에 속한 스타일을 class로 덮어쓸 수 없어 유지보수와 확장성에 불리할 수 있어요.
- id는 페이지 내에서 고유해서 재사용에 좋지 않지만, class는 재사용 가능해요.
text-align: center; | ||
line-height: 26px; | ||
color: #ffffff; | ||
transition: all 0.2s; |
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.
transition 설정 좋아요~👍
} | ||
|
||
a { | ||
text-decoration: none; |
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.
브라우저에서 기본 제공하는 스타일링 제거 목적인 경우 최상단에 따로 모아두거나 초기화 할 설정이 많아지면 파일을 분리해서 관리해도 좋아요.
line-height: 56px; | ||
} | ||
|
||
.buyBtn { |
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.
클래스, id 이름으로 camelCase, snake_case 혼용해서 사용하고 있는데 둘 중 하나로 통일해서 작성하면 코드 읽는데 더 좋을 것 같아요.
grid-template-columns: minmax(200px, 1fr) minmax(400px, 1120px) minmax( | ||
200px, | ||
1fr | ||
); |
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.
minmax 줄바꿈이 어색해요
<title>판다마켓</title> | ||
</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.
다른 페이지로 이동하기 쉽도록 상단에 링크를 두는 요소는 nav태그가 더 좋을 것 같아요.
header태그는 콘텐츠의 제목이나 소개 또는 탐색하는 영역에 사용해요.
3c2b747
into
codeit-bootcamp-frontend:Basic-남기연
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게
https://pandamarket-namgyeon.netlify.app/ 네트라파이 주소입니다.