Conversation
| @@ -0,0 +1,109 @@ | |||
| <!DOCTYPE html> | |||
| <html lang="en"> | |||
There was a problem hiding this comment.
lang 속성값으로 en 을 지정해주셨네요.
lang 은 사용할 언어를 정의하는 것이고 한국어로 디자인된 사이트를 작업하고 있으니
한국어를 나타내는 값으로 수정해야 합니다.
찾아보시고 수정해주세요!
| <html lang="en"> | ||
| <head> | ||
| <meta charset="UTF-8"> | ||
| <meta http-equiv="X-UA-Compatible" content="IE=edge"> |
There was a problem hiding this comment.
! 로 입력하는 템플릿 코드를 사용할 땐
어떤 설정을 의미하는 지 확인 후 필요에 맞게 사용해주세요.
| <title>Linkbrary</title> | ||
| </head> | ||
| <body> | ||
| <div class="gnb"> |
There was a problem hiding this comment.
불필요하게 중첩된 태그를 정리하고 그에 맞춰 스타일링도 수정해주세요.
| </head> | ||
| <body> | ||
| <div class="gnb"> | ||
| <div class="frame-1237"> |
There was a problem hiding this comment.
class 명도 좀 더 의미있게 작성해서 어떤 부분인지 실행시켜보지 않아도 파악할 수 있게끔 수정해 주세요.
| <div class="frame-1237"> | ||
| <div class="logo"> | ||
| <a href="/"> | ||
| <img src="src/Linkbrary.png" alt="Linkbrary Logo"> |
There was a problem hiding this comment.
alt 속성 값을 잘 입력해주셨습니다.
멘토링 때 말씀드린 것처럼 이미지 사이즈 설정해주세요!
| <div class="frame-2608782"> | ||
| <div class="frame-2608745"> | ||
| <div class="policy"> | ||
| <img src="src/policy_faq.png" width="(87px + 29px + 30px)" height="20px"></img> |
There was a problem hiding this comment.
width, height 속성값에는 스펙 상 숫자만 가능하다는 점 참고해 수정해 주세요.
| <img src="src/content1.png" alt="첫번째 컨텐츠"> | ||
| </div> | ||
| <div class="link"> | ||
| <img src="src/add_link.svg" alt="링크 추가하기"> |
There was a problem hiding this comment.
이런 버튼 요소도 직접 구현해주세요.
| /* landing/pc */ | ||
|
|
||
| /* Auto layout for HTML */ | ||
| html { |
There was a problem hiding this comment.
html 태그는 head 와 body 태그를 감싸고 있는게 일반적이므로 스타일링을 하기엔 조금 적절하지 못하다고 생각됩니다.
다른 적절한 선택자을 찾아 스타일링 해주시면 좋을 것 같습니다.
| .gnb { | ||
| position: absolute; | ||
| display: flex; | ||
| flex-direction: column; |
There was a problem hiding this comment.
요것도 멘토링 때 말씀드린 것처럼 피그마의 속성값을 참고하기 보단
해당 디자인을 구현하기 위해서 필요한 속성만 추가해서 작업하시면 좋을 것 같습니다.
| top: 0; | ||
| left: 0; | ||
| right: 0; | ||
| z-index: 1000; /* 다른 요소 위에 표시되도록 설정 */ |
There was a problem hiding this comment.
z-index 값도 신중하게 설정해야 한다는 점도 구두로 말씀드렸습니다!
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게