-
Notifications
You must be signed in to change notification settings - Fork 79
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
[고재성] Sprint1 #50
The head ref may contain hidden characters: "part1-\uACE0\uC7AC\uC131-sprint1"
[고재성] Sprint1 #50
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.
정말 깔끔하고 가독성좋게 코드 잘 짜시는것같아요.
Approve! 너무 잘하셨지만, 개념정리 & 참고될만한 코멘트 추가로 남겨드려봤어요
재성님 수고하셨습니다~!! 👍
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.
폴더구조도 그렇고 파일 위치, 용도 적절히 잘 이해하고 사용해주신것같아요! 퍼펙트 👍
} | ||
:root { | ||
/*basic colours*/ | ||
--gray900: #1b1d1f; |
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.
css vars를 사용해 자주 사용하는 컬러코드를 만들어놓으면 반복되는 코드도 줄이고 변경에도 유연하게 대처할수있겠죠? 잘하셨습니다 👍
a:hover { | ||
cursor: pointer; | ||
} | ||
.btn { |
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.
요런 버튼의 경우 primary button이라는 세부항목을 추가해서 공용화해주는게 더 좋을것같네요 :)
네이밍의 경우 BEM 규칙 참고해서 진행해보세요! ㅎㅎ
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.
깔끔하게 잘 작성하시고, 네이밍만으로도 구조도 한눈에 보여서 정말 가독성이 좋네요!!
<meta charset="UTF-8" /> | ||
<meta name="viewport" content="width=device-width, initial-scale=1.0" /> | ||
<title>판다마켓</title> | ||
<link rel="stylesheet" href="css/reset.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.
스타일시트 적용순서에 따라 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.
https://blog.naver.com/hahav000/221542970686
이 글 참고해보시면 개념 정리하는데 도움이 될것같네요!
</div> | ||
</section> | ||
<section class="featured-container"> | ||
<div class="featured-box"> |
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.
container > box같이, 일반적으로 nesting된 div를 표현할때 사용되는 네이밍을 사용해서 구조가 더 한눈에 들어오는것같아요. 잘하셨습니다! 이런 방법말고도 feature-container-item등과 같이 네이밍하는 방법도 있답니다 :) 네이밍은 세부적으로 어떻게 쓰냐보다, 어떤 반복되는 규칙과 구조를 가지고 통일감있게 썼는지가 중요한거라서 이런 기술스택을 쓰는경우 보통 어떻게 네이밍하는지 찾아보시는것도 좋은 연습이 될것같아요 :)
멘토의 답변배너에 있는 이미지를 하늘색 배경화면의 밑부분과 맞추기 위해 'position:absolute, bottom:0'을 사용했는데 다른 해결방식이 있을까요?좋은 질문입니다! ㅎㅎ 지금과 비교해서 더 나은 방식이 있을지 고민하는 마인드셋은 성장에 큰 도움이 된답니다 :) |
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게