Conversation
There was a problem hiding this comment.
과제하느라 고생하셨습니다! 질문 답변도 같이 남겨둘게요!
-
css를 작성할 때 가독성과 유지보수에 좋게 작성하는 방법이 궁금합니다!
가독성이 좋으면 유지보수할때도 편해지기 때문에 둘 다 관계가 있다고 생각하는데요. 과제 해주신 것을 보니, 가독성 좋게 잘 만들어주셨습니다 ㅎㅎ 한 가지 팁을 드리자면 승래님만의 규칙을 만들어서 작성해보시는 것을 연습해보셔도 좋습니다. 크기 관련된 부분과 패딩, 마진 부분을 구분해준다던지, 순서를 일정하게 작성해본다던지 등이요! 그렇게 규칙이 생기면 가독성에 좀 더 도움이 되겠죠? -
랜딩 페이지의 url path는 루트(‘/’)설정에 대해 href="index.html"으로만 해도 괜찮은지 궁금합니다!
경로라는 표시를 해주기 위해 앞에 / 를 붙여서 /index.html로 써주셔도 좋을것 같습니다. 하지만, 지금처럼 작성하셔도 크게 상관은 없습니다!
| @@ -0,0 +1,11 @@ | |||
| <!DOCTYPE html> | |||
| <html lang="en"> | |||
There was a problem hiding this comment.
lang="ko"로 써주신곳도 있는것 같은데, 아예 ko로 통일해주시면 좋을것 같아요!
There was a problem hiding this comment.
이미지의 파일명은 소문자로 써주시면 좋습니다!
| <img src="images/Img_home_01.png" alt="메인_섹션_이미지_1" /> | ||
| <div class="section_container"> | ||
| <h3>Hot item</h3> | ||
|
|
There was a problem hiding this comment.
저는 태그간의 구분을 위한 개행은 굳이 하지않는 편입니다. 개행하지 않아도 다 알아볼 수 있기 때문이죠!
| @@ -0,0 +1,11 @@ | |||
| <!DOCTYPE html> | |||
| <html lang="en"> | |||
There was a problem hiding this comment.
요기는 특히 한글이 들어가서 더더욱 ko로 바꿔주시면 좋을것 같습니다.
| body { | ||
| padding: 0; | ||
| margin: 0; | ||
| } |
There was a problem hiding this comment.
body에 기본적으로 걸려있는 css를 초기화해주시려고 한것같네요! reset css에 대해서 검색해보시고 사용해보셔도 좋을것 같습니다.
| } | ||
|
|
||
| button:active { | ||
| background-color: #1251aa; |
There was a problem hiding this comment.
컬러 같은 경우는 변수로 따로 모아서 관리해주시면 좋습니다. 필요한 곳에서 꺼내쓸 수 있기 때문이죠!
| margin: 0 auto; | ||
| } | ||
|
|
||
| header a{ |
| width: 355px; /* 배너 구경하러가기 버튼 */ | ||
| height: 56px; /* 배너 구경하러가기 버튼 */ | ||
| border-radius: 40px; /* 배너 구경하러가기 버튼 */ |
There was a problem hiding this comment.
같은 내용의 주석이 반복되어 적혀있네요! 지워주시면 좋을것 같습니다.
|
|
||
|
|
There was a problem hiding this comment.
개행이 두번 되어있네요. 하나는 지워주시면 좋을것 같습니다.
| font-family: Pretendard; /* footer 공통요소 */ | ||
| font-size: 16px; /* footer 공통요소 */ | ||
| font-weight: 400; /* footer 공통요소 */ | ||
| line-height: 19px; /* footer 공통요소 */ | ||
| letter-spacing: 0em; /* footer 공통요소 */ | ||
| text-align: center; /* footer 공통요소 */ |
There was a problem hiding this comment.
공통요소들은 footer 전체에 한번만 적용해보시면 어떨까요? 코드를 한번만 작성해둘 수 있습니다. 아니면, 새로운 클래스로 만들어서 관리하시는것도 좋고요!
요구사항
기본
심화
스크린샷
멘토에게