Conversation
There was a problem hiding this comment.
main-bg1.png 정도면 더 명확하게 이미지의 역할을 파악할 수 있을 것 같습니다!
| @@ -0,0 +1,148 @@ | |||
| .body { | |||
| color: var(--black, #000); | |||
There was a problem hiding this comment.
--black, --linkbrary-bg 등 CSS 변수에 대한 선언이 빠졌네요,
기본값을 명시해주셔서 정상적으로 스타일이 적용되긴 하겠지만... 사용하고 계신 변수들은 선언 후에 사용해주세요.
| .social-icon { | ||
| width: 42px; | ||
| height: 42px; | ||
| position: relative; | ||
| width: 42px; | ||
| height: 42px; |
There was a problem hiding this comment.
width, height 선언이 두번씩 들어가 있습니다.
의도하신게 아니라면 불필요한 중복 코드는 제거해주세요!
| <header> | ||
| <nav> | ||
| <a href="index.html"> | ||
| <img src="./images/logo.svg" alt="홈으로 연결된 Linkbrary 로고" /> |
There was a problem hiding this comment.
파일 별로 상대 경로(ex. ./imagegs/logl.svg)와 절대 경로(ex. /imagegs/logl.svg)를 혼용하고 계신데요,
한 프로젝트 내에서는 한가지 방법으로 통일하는게 일관성, 가독성을 위해 좋습니다.
절대 경로/상대 경로에 대해 알아보시고 원하는 방법 한가지로 통일해서 사용해 주세요.
|
|
||
| position:fixed; | ||
| top: 0; | ||
| background-color: #edf7ff; | ||
| } |
There was a problem hiding this comment.
fixed 속성으로 nav 고정을 잘 구현해주셨네요 👍
| #icon-kakao { | ||
| width: 26px; | ||
| height: 24px; | ||
| flex-shrink: 0; | ||
| z-index: 1; | ||
| position: absolute; | ||
| top: 50%; | ||
| left: 50%; | ||
| transform: translate(-50%, -50%); |
There was a problem hiding this comment.
아이콘 중앙 정렬도 position, transform 속성을 이용해서 잘 구현해주셨습니다 👍
|
고생 많으셨습니다. |
요구사항
기본
성
심화
주요 변경사항
스크린샷
멘토에게