-
Notifications
You must be signed in to change notification settings - Fork 42
[김이서] sprint3 #141
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
The head ref may contain hidden characters: "Basic-\uAE40\uC774\uC11C-sprint3"
[김이서] sprint3 #141
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.
수고하셨습니다!
주요 리뷰 포인트
- 반응형 이미지 리소스 최적화
- 줄바꿈 속성 관련 피드백
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.
투명 배경을 지원할 필요가 없으면 png보다는 jpg 파일 사용을 권장드립니다.
추천 드리자면:
- 벡터 기반의 장점을 활용할수있거나, path 정보가 간단한 로고, 아이콘등의 파일: svg
- 투명 배경을 지원할 필요가 있는 큰 이미지 파일: png
- 투명 배경을 지원할 필요가 없는 큰 이미지 파일: 최소 2x의 고배율 jpg, jpeg => 고배율 이미지를 써야하므로 png, jpeg보다 압축률이 좋은 webp로 변환해 picture태그와 함께 사용 추천
컨텐츠의 특성 혹은 몇가지 조건에 따라 올바른 파일 포맷을 고르는것도 최적화에 포함됩니다.
제가 추천드린것말고도 여러가지 참고해보시고 적용해보세요 :)
<div class="feature-content"> | ||
<h2>Search</h2> | ||
<h1> | ||
구매를 원하는 <span class="break-on-desktop"><br /></span>상품을 |
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.
break-on-desktop 클래스를 따로 만들어 br태그를 표시하지않게끔 하는것보다는, 줄바꿈 문자를 넣어주고 white-space: pre-line
속성을 사용하거나 word-break: keep-all;
을 사용하는 방식은 어떨까요?
</div> | ||
<img src="img/Img_home_02.jpg" alt="검색하는 판다" /> | ||
<div class="feature"> | ||
<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.
여기 쓰인 이미지도 srcset과 sizes 사용을 고려한다면 반응형 이미지 리소스 최적화가 이루어질수있겠죠? 첫 화면에 보이는 이미지가 아닌 스크롤을 쭉 내려야 보이는 이미지라면 레이지로딩을 적용할수도있고요 :)
><img src="images/logo/logo.svg" alt="판다마켓 홈" | ||
/></a> | ||
|
||
<form method="post"> |
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.
굳굳! 접근성과 서버 데이터 제출을 고려해 잘 작업하셨네요 👍
} | ||
|
||
@media (min-width: 768px) { | ||
.auth-container { |
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 변수로 관리해보시는것도 좋을것같아요 :)
요구사항
기본
심화
주요 변경사항
스크린샷
배포사이트
https://exquisite-hamster-ca0014.netlify.app/
멘토에게