[송미진] 스프린트2 #28
[송미진] 스프린트2 #28
Conversation
dongqui
left a comment
dongqui
left a comment
There was a problem hiding this comment.
미진님 이번 미션도 잘 마무리 해주셨군요!!
고생 많으셨습니다~
form 관련 태그들을 올바르게 사용해주셨네요! 스타일 코드를 분리하고 계시는 김에 공통 부분을 따로 빼보시면 더욱 좋을 거 같습니다~!
링크, 이미지 등 절대경로에서 상대경로로 모두 수정했습니다.
그대로 사용해도 괜찮은가요.. 어떤것이 나은 방법인지 궁금합니다.
-> 개발 환경이나 배포 환경에 따라 달라지는데, 지금은 아무거나 하셔도 크게 상관 없습니다 :)
| .font_1 { font-size:40px; } | ||
| .font_2 { font-size:24px; } | ||
| .font_3 { font-size:18px; } | ||
| input:hover, |
There was a problem hiding this comment.
ui.css쪽으로 가는 게 어울릴 거 같네요 🤣
There was a problem hiding this comment.
해당 css파일은 이제 공통으로 사용되기 어려울 거 같습니다! 헤더나, 섹션, 푸터 모두 랜딩 페이지에만 적용되는 것들이고 다른 페이지에는 불필요한 스타일들이죠!
layout.css는 랜딩 페이지 스타일로 두고, 공통 부분을 따로 빼시는 것도 좋을 거 같아요 :)
| @@ -1,10 +1,30 @@ | |||
|
|
|||
| :root { | |||
| <img src="/images/Img_home_01.png" alt="Hot item"> | ||
| <img src="images/Img_home_01.png" alt="Hot item"> | ||
| <div> | ||
| <h4 class="font_3">Hot item</h4> |
There was a problem hiding this comment.
계층 구조를 다시 확인해 보시면 좋을 거 같아요 :)
https://developer.mozilla.org/ko/docs/Web/HTML/Element/Heading_Elements#사용_일람
h3 없이 h4가 나와서는 안 됩니다! 꼭 h tag를 안 쓰셔도 될 거 같기도 하구요!
| <a href="https://www.youtube.com/" target="_blank"><img src="/images/ic_youtube.png"></a> | ||
| <a href="https://www.instagram.com/" target="_blank"><img src="/images/ic_instagram.png"></a> | ||
| <div class="login_box"> | ||
| <form onchange="login_check()"> |
| <a href="https://www.youtube.com/" target="_blank"><img src="/images/ic_youtube.png"></a> | ||
| <a href="https://www.instagram.com/" target="_blank"><img src="/images/ic_instagram.png"></a> | ||
| <div class="login_box"> | ||
| <form onchange="login_check()"> |
There was a problem hiding this comment.
input value에 따라 버튼의 상태를 실시간으로 바꾸고 싶으신 거라면 change 대신 input 이벤트를 고려해보셔도 좋을 거 같아요~
| <a href="https://www.instagram.com/" target="_blank"><img src="/images/ic_instagram.png"></a> | ||
| <div class="login_box"> | ||
| <form onchange="login_check()"> | ||
| <label for="login_email">이메일</label> |
| </main> | ||
| <script> | ||
| function login_check(){ | ||
| let in_email = document.getElementById('login_email').value; |
There was a problem hiding this comment.
자바스크립트 코드에서는 주로 카멜케이스를 사용합니다 :) 참고해주세요!
그리고 이름을 잘 짓는 것은 언제나 중요합니다! 조금 길어지더라도 명확하게 써주시는 게 좋아요 :)
in_은 직관적이지 않은 거 같아요!
| <script> | ||
| function login_check(){ | ||
| let in_email = document.getElementById('login_email').value; | ||
| let in_pwd = document.getElementById('login_pwd').value; |
There was a problem hiding this comment.
재할당되지 않는 값들은 주로 const를 사용합니다!
|
감사합니다!! |
2 participants
요구사항
기본
심화
주요 변경사항
-이전에 주신 피드백 수정했습니다.
-미션1과 다르게 미션2작업하면서 폴더 기준을 잘못잡아서 안나오길래....
링크, 이미지 등 절대경로에서 상대경로로 모두 수정했습니다.
그대로 사용해도 괜찮은가요.. 어떤것이 나은 방법인지 궁금합니다.
스크린샷
멘토에게