-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[김성종] Spint1&2 #119
The head ref may contain hidden characters: "Basic-\uAE40\uC131\uC885-sprint1"
[김성종] Spint1&2 #119
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
잘 작성해주셨습니다.
- {} 부분은 전체 선택자이므로, 보통 클래스명, id명에 적용하는 것보다 더 포괄적인 부분에 적용되는 스타일이라고 생각해주시면 될 것 같습니다.
보통 button을 많이 쓰는 편인 것 같습니다. 하지만, 그렇다고 a태그를 사용하지 않지는 않습니다.
학습방법의 경우, 현재 많은 코드를 작성해보지 않으셔서, 더 그렇게 생각이 드시는 것 같습니다.
강의를 이해하시는 것도 중요하지만, 많이 작성해보시고, 하는 것도 큰 도움이 되기 때문에, 앞으로는 미완성이여도 과제를 조금이라도 작성해보시면서, 코드를 많이 작성해보시면 좋을 것 같습니다.
고생하셨습니다! 🙇🏻
<!-- 머리 --> | ||
<nav class="navbar"> | ||
<div class="navbar_logo"> | ||
<img src="/img_file/판다 얼굴.png" class="navbar_panda"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
img 태그에서 보통 alt 속성을 빼먹기 쉬워서, 잘 작성해주시면 좋을 것 같습니다.
그리고
<img > 단일 태그일 경우 아래와 같이 작성하는 편입니다.
<img />
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
</div> | ||
</footer> | ||
</body> | ||
</html> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
잘 작성해주셨습니다.
content별로 1,2,3,4,5로 css 클래스명으로 구분해주셨는데,
1,2,3,4,5보다는 조금더 명확한 네이밍이면 좋을 것 같습니다. 그 이유는 만약에 중간에 3번 컨텐츠를 지울 경우, 1,2,4,5로 남게 되는데, 이렇게 될 경우 코드 구조 파악이 어려울 것 같습니다
<label for="email" class="label_email">이메일</label> | ||
<input id="email" name="email" type="email" placeholder="이메일을 입력해주세요" required autocomplete="email"> | ||
</div> | ||
<div class="login_pw_bundle"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bundle이라는 의미는 어떤 의미에서 사용하신건지 여쭤보고 싶습니다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
label, input 태그 요소를 하나로 묶어 지칭할 만한 단어가 딱히 떠오르지 않아, package, bundle 등 고민하다가 임의로 했습니다!
</div> | ||
</div> | ||
</body> | ||
</html> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
잘 작성해주셨습니다.
img 태그의 alt 속성과 단일태그 부분만 잘 신경써주시면 좋을 것 같습니다.
그리고 시멘틱 태그 뿐 아니라, css 클래스명을 통해서도 구조 파악을 할 수 있으니, 해당 부분도 신경써주시면 좋을 것 같습니다
<meta charset="UTF-8"> | ||
<meta name="viewport" content="width=device-width, initial-scale=1.0"> | ||
<link rel="stylesheet" href="style_signup.css"> | ||
<title>Document</title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
singup 파일인데, title은 Document로 되어있는 것 같습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
감사합니다! 번외로 궁금한 게, 실무에서는 구현해야 하는 페이지별로 html 파일을 따로 만드나요??
저는 한 html 파일 내에 위 코드들을 어떻게 합쳐야 유려할지 / 가독성 좋을지 단번에 떠오르지 않기도 했고, 모듈의 개념이 이런 건가? 하며 따로 만들기는 했었습니다!
<label for="password" class="label_pw">비밀번호</label> | ||
<input id="password" name="password" type="password" placeholder="비밀번호를 입력해주세요" required> | ||
</div> | ||
<div class="signup_pw2_bundle"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pw2보다는 password_check같은 명확하게 알 수 있는 css명은 어떨까요 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네 class 네이밍은 아무래도 고민이 많이 필요할 거 같네요, 말씀주신 아이디어가 더 적절해보입니다!
[Sprint1]
요구사항
기본
심화
스크린샷
[Sprint2]
요구사항
기본
심화
스크린샷
멘토에게
ex) css 에서 * { } 안에 들어간 게 보통과는 어떻게 다른지, body 태그에 스타일링을 많이 하는 것보다 개별 지정 혹은 * { } 안에 하는지, sns 간편 로그인 버튼은 img 와 a 태그를 통한 링크 연결만 하는지(혹은 제가 작성한 것처럼 button 태그를 쓰는지) 등