-
Notifications
You must be signed in to change notification settings - Fork 2
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.
오우!!!!!은비니 시멘틱화 하려고 상당히 많이 노력하고 클래스명도 고민있게 작성해준게 보인다!!
사소하게 놓치는 속성들이 있었는데 그것들만 천천히 보완하면
탄탄한 웹개발자 될것같넹💛 고생해써!!
breadStore/main.html
Outdated
@@ -0,0 +1,111 @@ | |||
<!DOCTYPE html> | |||
<html lang="kr"> |
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.
kr은 오디언어래여?
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.
ㅎㅎㅎㅎㅎㅎㅎㅋㅋㅋㅋㅋㅋㅋㅋㅎㅎㅎㅎㅎㅎㅎㅎ 머쓱하게 수정...
breadStore/main.html
Outdated
<div class="main"> | ||
<nav class="nav"> | ||
<ul> | ||
<p class="nav-title">종류</p> |
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.
옹 디테일해,,,
breadStore/main.html
Outdated
<p class="title">은빈이네 빵집 🥐</p> | ||
<img class="menuBar" src="./imgs/7216128.png" /> | ||
</header> | ||
<div class="main"> |
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.
main으로 묶지않고 클래스명을 main으로 한 이유는 무엇일까여?
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.
그르게! 여기 바로 main으로 태그를 주고 클래스명은 주지 않아도 괜찮을 것 같은데!
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.
아마 main 태그 역할을 확실하게 확인하고 수정하려고 뒀다가 그냥 잊혀졌나봐... 👀..
시멘틱태그가 어색하다 ㅠ
breadStore/styles.css
Outdated
padding: 1em; | ||
background-color: #feffff; | ||
border-radius: 20px; | ||
box-shadow: 2px 2px #2b7a78; |
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.
var 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.
다음 과제 땐 이거 꼭 써볼거야
breadStore/main.html
Outdated
@@ -9,6 +9,7 @@ | |||
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin> | |||
<link href="https://fonts.googleapis.com/css2?family=Noto+Sans+KR:wght@100;300;400;500;700;900&display=swap" rel="stylesheet"> | |||
<link rel="stylesheet" href="styles.css" /> | |||
<link rel="icon" href="./imgs/BreadA.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.
앙웅!디테일해💛
breadStore/main.html
Outdated
<!-- ❤️ --> | ||
</div> | ||
<div class="card"> | ||
<p class="bakery-title"> 빵 이름 </p> | ||
<p class="bakery-tag">#hash #tag</p> | ||
<img class="bakery-img" src="./imgs/BreadA.png"/> | ||
<button class="bakery-like"> ♡ </button> | ||
<button class="bakery-like"> ♥ </button> |
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 type 지정왜하는지 찾아보면 조흘것가타
breadStore/styles.css
Outdated
text-decoration: none; | ||
} | ||
|
||
.category [href] { |
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.
오호!
breadStore/main.html
Outdated
<div class="menu-bar__detail"> | ||
<ul class="menu-bar__list"> 메뉴 | ||
<hr /> | ||
<li class="menu-bar__list-title"> 새 상품 추가 </li> |
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.
클래스명이 완전 야무진데
<li class="detail-title"> 찜 목록</li> | ||
</ul> | ||
</div> | ||
<div class="c-menu-bar"> ☰ |
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.
c는 무슨의미에여?
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.
그르게! c는 어떤 의미인가요?!
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.
정답! choice에서 가져온거 같다!
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에서 가지고 왔던건데 지금 보니 의미가 명확하지 않군요.... 다른 클래스명을 다시 생각해봐야겠으요 🤔
breadStore/main.html
Outdated
<li class="section__tag-list">#옆면이 보이는</li> | ||
<li class="section__tag-list">#바게뜨</li> | ||
<li class="section__tag-list">#바게뜨?</li> | ||
<li class="section__tag-list">#바게트?</li> |
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.
:before 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.
은빈이 첫 과제 너무 수고했어🌸
이것저것 시도한 게 보이고, 클래스명도 직관적으로 잘 지으려고 한 게 보인다!!
코드리뷰 보고 한 번 다시 쓱 살펴보면서 공부 다시 해보면 좋을 것 같아! ㅎㅎ
@@ -0,0 +1,196 @@ | |||
@import "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.
보통 style.css
로 명명하는데 styles.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하면 Cascading Style Sheets 가 같이 떠올라서 styles.css 라고 적는 듯?! 뭔가 나한테 스타일 시트하면 여러 개인 느낌이ㅇㅑ..
breadStore/styles.css
Outdated
@import "reset.css"; | ||
|
||
body { | ||
padding-top: 4em; |
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 속성에 별 다른 순서를 정하지 않은 것 같은데, 순서에 맞춰서 속성을 적어주면 가독성도 올라가고 깔끔해져서 한 번 시도해보면 좋을 것 같아! 내 PR을 한 번 봐도 좋고, 현업에서 사용중인 CSS 작성 순서와 쉽게 자동 정렬하기(VScode 확장프로그램) 이거 참고해봐도 좋을 것 같아!
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: flex; | ||
justify-content: space-between; | ||
align-items: center; | ||
color: #feffff; |
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 속성 사용하기 (변수)
<li class="detail-title"> 찜 목록</li> | ||
</ul> | ||
</div> | ||
<div class="c-menu-bar"> ☰ |
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.
그르게! c는 어떤 의미인가요?!
breadStore/main.html
Outdated
<body> | ||
<!-- header --> | ||
<header> | ||
<p class="title">은빈이네 빵집 🥐</p> |
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.
p 태그 말고 h1 태그를 이용해보는 건 어때?! 아무래도 페이지에서 가장 중요한 제목이니까! p를 사용하니까 우선순위가 많이 떨어져보여서 ㅎㅅㅎ
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.
h1~h6 태그에 손이 많이 안가서 거의 안썼는데 시멘틱 태그에서 이 태그들이 내용의 우선순위에도 유의미하단 걸 잊고 있었네!!! 리펙토링 할 때 참고해서 heading tag 부지런히 써봐야겠다
breadStore/styles.css
Outdated
.nav__category:hover { | ||
color:#def2f1; | ||
background-color: #17252A; | ||
text-decoration: 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.
text-decoration: none;
이것도 기본값이라서 빼도 될 것 같앙!
breadStore/styles.css
Outdated
text-decoration: none; | ||
} | ||
|
||
.nav__category [href] { |
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.
저도 덕분에 새로운 방법 배워 갑니다~!👍🏻
breadStore/styles.css
Outdated
} | ||
|
||
section { | ||
flex-grow: 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.
section_card
에 width가 지정되어 있고, flex-wrap
도 적용이 되어있어서 flex-grow
필요없을 것 같은데!!
width: 55vw; | ||
height: auto; | ||
} | ||
.section__tag { |
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.
구현 화면 보니까 max-width:650px
에서는 해시태그가 그대로 다 보여서 여기서도 따로 해시태그 컨테이너 width에 대해 처리를 해줘야할 것 같아!
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 보면서 디테일한 부분들을 많이 놓쳤구나 체감했었어 ㅋㅋ ㅠㅠ 참고해야겠어
breadStore/styles.css
Outdated
border-radius: 10px; | ||
display: flex; | ||
flex-direction: column; | ||
flex-shrink:0; |
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.
보니까 지금 nav에는 따로 너비를 주지 않고 nav-category
에 width: 6rem;
를 줘서 이 너비에 따라 nav의 너비도 맞춰진 상태인 것 같거든?! 그리고 height도 nav-category
에 지정된 margin, padding에 따라 조정되고 있는 상황이라서 flex-shrink:0;
를 안써도 될 것 같아! 원래 flex-shrink가 정해진 너비를 자식 요소가 벗어나면 너비에 맞춰서 얘를 줄이지 말고 정해진 크기를 유지하라는 건데, 부모 요소에 정해진 너비가 없고 자식 요소의 크기에 따라 조정되고 있잖아! 그래서 안써줘도 될 것 같아!
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.
은빈 첫 과제 진짜 고생 많았어!! 엄청 꼼꼼하게 고민하면서 만든 게 보인다✨
<!-- header --> | ||
<header> | ||
<p class="title">은빈이네 빵집 🥐</p> | ||
<div class="c-menu-bar"> ☰ |
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.
나는 아이콘 넣었는데 이렇게 그냥 텍스트로 넣어주는 게 더 깔끔하고 좋은 것 같네!
breadStore/main.html
Outdated
<div class="c-menu-bar"> ☰ | ||
<div class="menu-bar__detail"> | ||
<ul class="menu-bar__list"> 메뉴 | ||
<hr /> |
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.
이전에는 hr 요소가 단순 가로선을 의미했고 지금도 브라우저는 수평 가로선으로 나타내지만, HTML5에서는 주제의 흐름이 바뀌었다는 의미적인 용어로 쓰인다고 해서 참고하면 좋을 것 같아요!
breadStore/main.html
Outdated
<!-- main : except header --> | ||
<div class="main"> | ||
<nav> | ||
<p class="nav__title">종류</p> |
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.
p 태그를 쓴 이유가 궁금해요! 가능하다면 헤딩 태그를 써도 좋을 것 같아요~
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.
위에서도 말했는데 heading 태그를 안써버릇하다보니까 손이 너무 안가서 거의 p 아니면 div 태그를 썼던 것 같아!!
웹 내 우선순위 표기에 있어서도 헤딩태그를 쓰는 쪽이 나을 것 같아서 리펙토링때는 고려해서 적어보겠습니당 !!!
</nav> | ||
<!-- section : container of cards --> | ||
<section> | ||
<!-- article : a card * 12 --> |
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.
주석으로 이런 구조 설명 적어놓으니까 이해하기 편하고 좋네요
breadStore/main.html
Outdated
<li class="section__tag-list">#모르겟다</li> | ||
</ul> | ||
<img class="section__img" src="./imgs/Bread1.png" alt="Baguette_IMG01"/> | ||
<button class="section__like"> ♥ </button> |
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.
버튼 타입 힉힉...써쥬긔~
breadStore/styles.css
Outdated
text-decoration: none; | ||
} | ||
|
||
.nav__category [href] { |
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.
저도 덕분에 새로운 방법 배워 갑니다~!👍🏻
[ 1주차 기본/심화/생각 과제 ] WEB 💛 TO DO MATE
✨ 구현 기능 명세
기본 과제
header
,nav
,card section
으로 나누어 구현전체
,분류 1~3
총 4가지로 구현상품 제목
,해시태그 목록
,이미지
,하트 찜 누르기
필수심화 과제
생각 과제
🌼 PR Point
🥺 소요 시간, 어려웠던 점
🌈 구현 결과물