-
Notifications
You must be signed in to change notification settings - Fork 31
[최권진] Sprint2 #45
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
[최권진] Sprint2 #45
The head ref may contain hidden characters: "Basic-\uCD5C\uAD8C\uC9C4"
Conversation
dongqui
left a comment
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.
권진님 이번 미션도 깔끔하게 잘 마무리 해주셨네요!
form 관련 태그를 잘 활용해 주셨습니다! 공통 스타일을 따로 정의해 보시면 더욱 좋을 거 같아요 :)
심화 미션의 비밀번호를 확인할 수 있는 아이콘을 추가 했는데, 다른 뷰포트에서 깨질 것 같다는 생각이 들었습니다. 다양한 뷰포트에서 테스트를 해볼 수 있는 좋은 방법이 있는지 궁금합니다.
->

개발자 도구에서 표시된 아이콘을 눌러보세요! :)
판다마켓 로고 위에 여백을 생성할 때 div태그로 height지정하는 방식과 margin으로 여백을 주는 방식 중 어떤 방식이 더 좋은 방식인지 궁금합니다.
-> 불필요하게 추가적인 dom 요소가 만들어지기 때문에 margin이 좀 더 적절해 보입니다 🤔
css 파일을 작성할 때 html코드 순서대로 css 선택자를 정의하면 협업 시 도움이 되나요? 실제 협업에서 css 코드를 html 태그 순서대로 작성하시는지 그냥 순서 상관 없이 작성하시는지 궁금합니다.
->
실제 협업에서는 지금처럼 페이지 단위로 css파일을 만드는 경우가 많지는 않아서 순서를 많이 신경 쓰지는 않았습니다! 조금 더 작은 단위로 스타일 파일을 만들어서 자연스레 코드가 나눠지게 됩니다 :) 스타일 라이브러리를 사용하면서 css파일을 안 만들 때도 있구요!
다만 순서를 맞추거나, 같은 목적을 가진 코드를 모으는 등 코드를 구조화 하는 노력은 많은 경우 도움이 됩니다!
| @@ -0,0 +1,142 @@ | |||
| :root { | |||
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.
변수를 잘 정의 해주셨네요 👍
| } | ||
|
|
||
| .eyes { | ||
| width: 20.47px; |
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.
피그마를 다시 확인해 주세요! 소수점까지 이용하는 경우는 거의 없습니다 :)
| right: 50px; | ||
| } | ||
|
|
||
| #google { |
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.
reset css를 활용해 보셔도 좋아요 :)
| @@ -0,0 +1,143 @@ | |||
| :root { | |||
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.
공통 스타일은 파일을 따로 빼서 관리하시면 좋습니다 :)
| </a> | ||
| <div class="main"> | ||
| <form action="/" method="POST" class="form"> | ||
| <label for="useremail">이메일</label> |
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을 잘 활용해 주셨네요! 👍
| <nav class="nav-flex"> | ||
| <a href="/" class="logo" | ||
| ><img | ||
| src="./assets/panda-logo.png" |
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.

요구사항
기본
심화
주요 변경사항
멘토에게