Conversation
There was a problem hiding this comment.
오 태훈님 js 로 토글도 동작하게 만들어주셨군요! 너무 좋은데요?
| <div class="section-box"> | ||
| <img class="section_content" src="images/Img_home_01.png" alt="Hot item 그림"> | ||
| <div class="section_content_p_box"> | ||
| <p class="section_content_p1">Hot item</p> |
There was a problem hiding this comment.
css 클래스이름을 엄청 꼼꼼하게 잘 작성해주셨네요.
한가지 아쉬운점은 어떤것은 kebab-case 로 작성되어있고 어떤건 snake_case 로 작성되어 있는 점이에요.
둘 중 어떤게 옳다! 라기보다는 하나로 맞춰서 일관성 있게 작성되면 좋을 것 같아요 :)
| } | ||
|
|
||
| :root { | ||
| --primary-color100: #3692FF; |
There was a problem hiding this comment.
오 컬러팔레트도 잘 이용해주셨군요 너무 좋습니다 :)
| .section-container-right { | ||
| max-width: 1920px; | ||
| height: 985px; | ||
| padding: 190px 200px 190px 200px; |
There was a problem hiding this comment.
이런경우에는 padding: 190px 200px 요렇게 축약해서 나타낼 수 있었을것 같아요
|
|
||
|
|
||
| :root { | ||
| --primary-color100: #3692FF; |
There was a problem hiding this comment.
css variant 를 설정해주는곳이 두군데가 되었어요. 이 부분만 별도로 분리해서 css 를 import 하게 하는건 어떨까요?
| label { | ||
| font-size: 18px; | ||
| font-weight: 700; | ||
| font-family: 'pretendard'; |
There was a problem hiding this comment.
font-family 는 별도의 이유가 없다면 최대한 가장 위! root 에서 공통으로 적용되도록 해주시는게 좋습니다 :)
| </a> | ||
| </div> | ||
| <div class="header-login-box"> | ||
| <a href="login.html"> |
There was a problem hiding this comment.
login.html 로 보내는것도 방법이지만, 멘토링때 이야기했던 것 처럼 login page 를 만들어서 /login 으로 보내고 해당 폴더의 index.html 을 불러오게끔 하는것도 좋은 방법입니다!
| } | ||
|
|
||
| .footer-link { | ||
| margin: 0px 10px 0px 10px; |
There was a problem hiding this comment.
margin 도 동일하게 margin: 0px 10px 이렇게 표현할 수 있을 것 같네요!
JaeSang1998
merged commit dcaabd3
into
codeit-bootcamp-frontend:Basic-김태훈
요구사항
Github에 PR(Pull Request)을 만들어서 미션을 제출합니다.
Netlify에 파일 배포가 아닌 포크한 Github 레포지토리로 연결합니다.
피그마 디자인에 맞게 페이지를 만들어 주세요.
React와 같은 UI 라이브러리를 사용하지 않고 진행합니다.
기본
로그인 페이지, 회원가입 페이지 공통
로그인 페이지
회원가입 페이지
심화