-
Notifications
You must be signed in to change notification settings - Fork 1
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
[1주차 기본/심화 과제] 웨비의 사진관 😋 #1
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.
너무나도 이쁘고 깔끔하게 잘 만드신 거 같아 사이트 구경하는 게 재밌었습니다!! 사진도 너무 이쁘게 잘 찍으시는 거 같아요~~
저의 짧은 지식을 총동원하여 리뷰 남겼습니다,, ㅎㅎ 제가 틀린 부분이 있을 수도 있으니 잘 걸러서 들어주세요!!
마지막으로, 승희가 저희 노션에 css 순서 관련해서 글 올렸는데 여기에 링크 달아놓을게요! 좋은 글인 거 같아서요
@@ -0,0 +1,113 @@ | |||
<!DOCTYPE html> | |||
<html lang="en"> |
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.
r u American?
} | ||
|
||
html, | ||
body { |
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과 body 둘 다에 font-size와 line-height을 주신 이유가 따로 있으신가요? 제가 알기론 font-size는 html 요소인 걸로 알고 있어서요
참고 자료 : https://stackoverflow.com/questions/48451764/how-to-set-base-size-for-rem
} | ||
|
||
/* third section */ | ||
.third-section-gallery { |
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.
세번째 섹션은 반응형이 아니라서 width 고정해 두어야 하는 건 아닌가 싶습니다!
아 그리고 다른 section이나 footer 이런 부분을 봐도 height만 설정 되어있고, width를 잘 설정 안 해놓으셨던데, 혹시 안 해도 무관한 부분일까요?? 결과물이 너무나도 깔끔하게 잘 나와서 없어도 상관 없는 건지 살짝 궁금합니다!
bottom: 3rem; | ||
z-index: 2; | ||
|
||
cursor: pointer; |
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.
화살표(▴) 부분에는 cursor: pointer; 가 적용되지 않고 있어요
확인부탁드려요~
header { | ||
padding: 0.7rem 0.5rem; | ||
margin-bottom: 2rem; | ||
width: fit-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.
우왕 이런 게 있었는지 몰랐어요!! 배워갑니다 :)
color: var(--gray-color); | ||
|
||
border-bottom: 0.2rem solid var(--gray-color); | ||
border-style: none none dotted none; |
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.
border-bottom: 0.2rem dotted var(--gray-color);
가 아니라
border-bottom: 0.2rem solid var(--gray-color);
border-style: none none dotted none;
이런 식으로 2개 나눈 이유가 있을까요?
z-index: 1; | ||
} | ||
|
||
.halftone { |
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로 하신 거군요! 너무 이쁩니다 😄
|
||
display: grid; | ||
gap: 1rem; | ||
grid-template-rows: 25% 25% 25% 25%; |
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.
repeat()를 사용하시면 더 간결하게 작성하실 수 있을 거 같아요!
gap: 1rem; | ||
} | ||
|
||
@media (max-width: 879px) { |
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.
저도 무대뽀 식으로 작성을 했었어 가지고요ㅜ
은서님 코드 보면서 제 코드 좀 수정하고 혹시라도 다른 방법 찾게 되면 다시 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.
진짜 너무 이쁘게 잘하신 것 같아요 ㅎㅎ
저 정말 너무 말하는 감자라....할 수 있는 게 감탄 밖에 없는 점 사죄드리고 분발하겠습니다 🥹🥹
margin-bottom: 2rem; | ||
width: fit-content; | ||
|
||
border-bottom: 0.4rem solid var(--green-color); |
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.
전 지정한 색 계속 찾아다니면서 rgb값 복사했는데 ..ㅎ var함수를 쓰면 편리하게 쓸 수 있군요!
.first-section-gallery > img { | ||
flex-grow: 1; | ||
flex-shrink: 1; | ||
} |
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.
전 첫 번째 섹션도 미디어쿼리를 이용했는데 flex-shrink 써서 다시 도전해보겠습니다!
grid-area: item4; | ||
} | ||
.grid-item:nth-child(5) { | ||
grid-area: item5; |
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.
grid 구현할 때 저는 아이템마다 row와 column을 줬는데 grid-template-areas를 이용하면 훨씬 편리하군용
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.
분위기 미쳐따
배포링크
✨ 구현 기능 명세
기본 과제
심화 과제
💎 PR Point
시맨틱 태그
header
,section
,nav
,main
,footer
태그를 사용 했습니다. preview 부분은 header와 section으로 이루어진 div로 구성하고, 그 밑의 카테고리명은 nav로, 그 밑의 세가지 이미지 갤러리는 각각 section으로 구성한 뒤 main으로 감쌌습니다. 만들고보니 li태그도 충분히 활용할 수 있었는데 존재를 잊었네요...! 그리고 뭔가 article 태그를 써볼까 했는데 아직 어디에 써야하는지 잘 감이 안오는 것 같습니다.첫 번째 섹션 반응형
display: flex
와flex-wrap: wrap
으로 기본적인 틀을 구현하고, 이 부모 아래 자식들 즉 모든 이미지에flex-grow: 1
을 부여하니 4 4 2 배치가 됐을 때 자동으로 마지막 줄의 두 이미지가 남는 영역을 모두 차지하게 할 수 있었습니다. 추가로flex-shrink: 1
을 사용해 창 너비가 줄어듦에 따라 이미지의 너비도 줄어들도록 했습니다. 하마터먼 media query로 구현할 뻔 했는데... YB 도윤님과 OB 재훈님의 도움을 받아 flex-grow라는 걸 난생 처음 써보게 되었어요 🙏두 번째 섹션 반응형
세번째 섹션 그리드
grid-template-areas
를 활용했습니다. 처음에는grid-template
으로 구현을 하려고 했는데 areas로 구성하는게 더 직관적이고 편하더라구요..!🥺 소요 시간, 어려웠던 점
8h
🌈 구현 결과물
배포링크