-
Notifications
You must be signed in to change notification settings - Fork 35
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 #73
The head ref may contain hidden characters: "Basic-\uC625\uC2B9\uD604-sprint1"
[옥승현]sprint1 #73
Conversation
<button class="headline-button">구경하러 가기</button> | ||
</a> | ||
</div> | ||
</section><!--headline END--> |
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.
어떤 값이 끝나는 지점에 주석은 굳이 불필요합니다!
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.
승현님 리뷰가 많이 늦었습니다 ㅠㅠ
컨텐츠가 부족한 페이지들은 꼭 채워보시면 좋을 것 같아요!
+) javascript 를 꼭 써서 동작을 추가해보시는걸 추천드립니다!
고생하셨습니다 :D
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 href="/" > | ||
<img class="logo-icon" src="./images/logo/logo.png" alt="판다마켓 로고"> | ||
</a> | ||
<a href="./login.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.
태그 안에 태그를 중첩해서 사용하지 않고, 버튼에 클릭 이벤트를 추가하는 방법도 있습니다! 혹은 button 에 as 등을 쓰는 방법도 있어요 :)
color: #E5E7EB; | ||
} | ||
|
||
@media(min-width: 1200px) and (max-width: 1920px){/* 데스크톱 뷰포트 설정*/ |
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.
반응형 디자인을 고려해주셨군요 :) 미디어 쿼리는 이해하고 있으면 나중에 큰 도움이 될거에요!!
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 는 한 파일로 되어있는데, 공용과 각각 페이지에 필요한 css 를 따로 분리해서 관리하면 나중에 유지보수, 확장성에 더욱 용이합니다!
</div> | ||
</section><!--headline END--> | ||
|
||
<section class="content"> |
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 구획을 semantic 하게 잘 나눠주셨네요! 의미 있는 태그로 분리하는건 아주 좋은 습관입니다.
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.
감사합니다! 수정해서 보완해보겠습니다!
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게
-늦게 제출해서 죄송합니다.