Conversation
urjimyu
left a comment
There was a problem hiding this comment.
세미나에서 배운 내용 적용하려는 고민들이 곳곳에 보여요👍🏻
시험기간이라 바쁠 텐데 심화과제까지 추가한 거 최고..!
정말 고생 많았어요!!🫶
Session01/htmlcss1/album.html
Outdated
| @@ -12,7 +12,7 @@ | |||
| <title>김밍밍</title> | |||
Session01/htmlcss1/style.css
Outdated
| background-color: palegoldenrod; | ||
| color: black; | ||
| text-align: center; | ||
| line-height: 2; |
There was a problem hiding this comment.
혹시 모두 line-height: 2;로 지정해준 이유가 있을까요?
그리고 line-height 속성에 대해 찾아보다가 생각과제였던 웹 접근성과 관련된 내용이 있어 공식문서 링크도 첨부합니다!
line-height
난독증 환자나 시력이 좋지 않은 사람들을 위해 line-height를 최소 1.5로 정하는 걸 추천해준다고 하는데 그런 면에서 좋은 코드라고 생각되어요:)
Session01/htmlcss1/style.css
Outdated
| top:0; | ||
| z-index:1; | ||
| } | ||
| /*position: sticky;*/ |
There was a problem hiding this comment.
저도 주석을 실수로 올리는 경우가 많은데요..!ㅎㅎ 필요하지 않은 주석은 제거하고 PR 올려주는 게 좋습니다!
Session01/htmlcss1/style.css
Outdated
| @@ -21,7 +25,8 @@ body { | |||
| } | |||
|
|
|||
| .prSction{ | |||
There was a problem hiding this comment.
CSS 네이밍 규칙에 맞게 줄임말보다는 직관적이고 이해하기 쉬운 클래스명을 사용하면 나중에 더 복잡해졌을 때 작업하기 수월해요!
저도 수정할 예정입니다..!ㅎㅎ
CSS 네이밍 규칙
Session01/htmlcss1/style.css
Outdated
| overflow-x: auto; | ||
| overflow-y: hidden; | ||
| display: flex; | ||
| width: 100%; height: 100%; |
There was a problem hiding this comment.
코드 정렬할 때 참고하면 좋은 자료 첨부합니다! 다만 너무 엄격하게 전부 지키려고 하면 오히려 효율이 떨어질 수 있어서 본인에게 맞는 방식으로 큼직하게 나눠주면 나중에 리팩토링할 때나 코드리뷰할 때도 편리하답니다😊
CSS 작성 순서
Session01/htmlcss2/account.html
Outdated
| <div class="description">고모부</div> | ||
| <div class="amountP">+ 250,000원</div> | ||
| <button class="deleteBtn">x</button> | ||
| <div class="deleBtn"> |
Session01/htmlcss2/account.html
Outdated
| <div class="items"> | ||
| <div class="category">식비</div> | ||
| <div class="description">스타벅스</div> | ||
| <div class="amountM">- 5,000원</div> |
There was a problem hiding this comment.
::before를 활용하면 한 번에 부호를 붙여줄 수 있습니다! 나중에 2주차 과제할 때 유용할 것 같아서 슬쩍 달아봅니다!
Session01/htmlcss2/account.html
Outdated
| </div> | ||
| <footer> | ||
| <div class="footerContent"> | ||
| <button id="add"><div style="cursor:pointer;">추가</div> |
There was a problem hiding this comment.
스타일은 css 파일을 만들어 놓았으니 css 파일들에 몰아 넣으면 더 좋을 것 같아요✨
There was a problem hiding this comment.
이전 커밋에 있는 애들이 리뷰가 한 줄씩 안 달리나봐요..! lang 설정이 영어로 되어 있어서 ko로 수정하는 게 좋을 것 같습니다!
Session01/htmlcss2/account.html
Outdated
| @@ -11,14 +11,14 @@ | |||
| <title>김밍밍</title> | |||
There was a problem hiding this comment.
title은 SEO를 고려해서 웹사이트 주제와 관련 있는 제목으로 수정하면 좋을 것 같아요:)
title 태그의 중요성
Session01/htmlcss1/album.html
Outdated
| <h2>NewYork</h2> | ||
| <header>NewYork</header> | ||
| <div class="container"> | ||
| <section class="prSction"> |
There was a problem hiding this comment.
저도 지민언니와 같이 img 태그 같은경우 div 로 감싸지 않고 바로 단독으로 썼으면 좋겠어요!
불필요한 div 태그를 남발하는 경우, 1. 불필요한 DOM node 생성으로 불필요한 메모리를 잡아먹는 단점이 있으니 div 태그의 필요성에 대해 고민하며 코드를 작성하면 좋을 것 같습니다! 👍
Session01/htmlcss1/album.html
Outdated
| <h2>NewYork</h2> | ||
| <header>NewYork</header> | ||
| <div class="container"> | ||
| <section class="prSction"> |
There was a problem hiding this comment.
또한 저도 고칠 점인데 생각과제로 알아본 웹 접근성을 위해서 ... 시맨틱 태그 적극적으로 활용하기!
<article>는 독립적으로 배포 및 재사용할 수 있는 독립형 콘텐츠에 사용되니 이 태그도 사용하면 좋을 것 같습니다.
Session01/htmlcss1/album.html
Outdated
| <footer> | ||
| <div class="footerContent"> | ||
| <p>© 2023 Sopt</p> | ||
| <button id="topButton"><div style="cursor:pointer;" onclick="window.scrollTo(0,0);">Top</div> |
There was a problem hiding this comment.
인라인 스타일을 사용했는데, 일반적으로 문제를 더 효과적으로 분리하고 유지 관리하기 위해 외부 CSS 파일에서 이러한 스타일을 정의하는 것이 좋습니다!
저는 <a href > 로 script 없이 이동시켜봤어요!
| @@ -0,0 +1,27 @@ | |||
| .sec01img{ | |||
| justify-content: center; /* 이미지를 가운데 정렬 */ | |||
| align-items: center; | |||
There was a problem hiding this comment.
이 속성같은 경우에 display: flex; 가 있어야 적용되는 속성입니다!
| } | ||
| .flexbox01{ | ||
| flex-wrap:wrap; | ||
| white-space: nowrap; |
There was a problem hiding this comment.
white-space 속성은 줄바꿈 및 공백을 설정합니다. 그래서 flexbox01 에는 불필요해 보이는 요소 같아요!
| border: 1px solid #ccc; /* 그리드 아이템 테두리 스타일 설정 */ | ||
| text-align: center; | ||
| } | ||
| .grid-item:first-child { |
Session01/htmlcss2/header.css
Outdated
| margin: 0px 0px; | ||
| } | ||
| /* Top 버튼 스타일 */ | ||
| .footerContent{ |
There was a problem hiding this comment.
bottom: 0; left: 50%; position: fixed; transform: translateX(-50%)
저번에 민영이가 물어본 footer 맨 아래에 위치하고 싶다 했는데 이 속성값들 추가시켜주면 height에 상관없에 맨 밑에 위치 시킬 수 있는 것 같아!
Session01/htmlcss2/account.html
Outdated
| </div> | ||
| </div> | ||
| <div class="sec01"> | ||
| <button id="prevBtn"><b><</b></button> |
There was a problem hiding this comment.
< 이 기호와 관련되어선 특수기호를 써주는게 좋을 것 같아!
< |
< | 꺽쇠 |
|---|---|---|
> |
> | 꺽쇠 |
|
시험기간인데 진짜 열심히 했다 미뇽아!! 민영이에게 참고가 될 것 같아서 html 구조 잡기 링크남겨! |
seobbang
left a comment
There was a problem hiding this comment.
과제 고생 많았어요오!ㅎㅎ 리팩토링까지 화이팅 🔥🔥
Session01/htmlcss1/album.html
Outdated
| <title>1차세미나 사진관 과제</title> | ||
| </head> | ||
| <body> | ||
| <header>김밍밍 인 뉴욕</header> |
There was a problem hiding this comment.
header 는 제목, 로고, 검색 폼, 작성자 이름 등의 요소 를 포함하는 소개 및 탐색에 도움을 주는 콘텐츠를 담는 영역!이기 때문에 정확히는 이 안에서
| <header>김밍밍 인 뉴욕</header> | |
| <header><h1>김밍밍 인 뉴욕</h1></header> |
이렇게 써주는게 좋답니다! :)
| <header>김밍밍 인 뉴욕</header> | ||
|
|
||
| <!--preview--> | ||
| <section class="preview"> |
There was a problem hiding this comment.
header,footer 제외하고 main으로 감싸주면 좀 더 시맨틱하겠죵 !
Session01/htmlcss1/album.html
Outdated
| <img src="wk1/02.png" alt="Photo"> | ||
| <img src="wk2/02.png" alt="Photo"> | ||
| <img src="wk3/01.png" alt="Photo"> | ||
| <img src="wk3/02.png" alt="Photo"> |
There was a problem hiding this comment.
alt 속성은 이미지가 보이지 않더라도 어떤 이미지인지 알 수 있도록 자세하게 써주셔야 웹 접근성에 좋답니당 !
| <section class="section-1"> | ||
| <h4 id="category-1">뉴욕은 낭만이 넘치는 도시에요</h4> |
There was a problem hiding this comment.
section-1, category-1보다는 각 내용에 맞게 의미를 가진 이름으로 명명해볼까요 ?! 의미를 띈 이름으로 class, id를 명명해주어야 추후 유지보수와 가독성에 좋은 습관이랍니당
ex) newyork-section
| <!--category1--> | ||
| <section class="section-1"> | ||
| <h4 id="category-1">뉴욕은 낭만이 넘치는 도시에요</h4> | ||
| <div class="images-1"> |
| <div class="borderline"></div> | ||
| <section class="summary-section"> | ||
| <article class="summary-article"> | ||
| <h3>나의 자산</h3> |
There was a problem hiding this comment.
heading태그는 h1부터 순차적으로 써주는게 원칙이랍니당!
| <button type="button"><b><</b></button> | ||
| <span id="currentDate">10/13</span> | ||
| <button type="button"><b>></b></button> |
There was a problem hiding this comment.
지민언니가 가영이 리뷰에 남겨준 것처럼, b태그는 최대한 지양하고 font-weight으로 스타일링해주는게 좋답니당 !
특수기호랑 type 넣어준거 넘 센수쟁이 ~~
Session01/htmlcss2/account.html
Outdated
| <button type="button"> | ||
| <button id="add">추가</div> | ||
| </button> |
There was a problem hiding this comment.
| <button type="button"> | |
| <button id="add">추가</div> | |
| </button> | |
| <button type="button">추가 | |
| </button> |
버튼이 중첩된건 어색한 구조에요! 이렇게 줘야하고, 그 안에서 뭔가 한번더 감싸야 하는 스타일링이 필요하다면 span 태그를 이용해봅시당
|
|
||
| .items { | ||
| display: flex; | ||
| flex-direction: row; |
There was a problem hiding this comment.
이것듀! default값이라 명시해주지 않아도 됩니당 !
Session01/htmlcss2/style.css
Outdated
| /* summary section | ||
| <section class="summary-section"> | ||
| <article class="summary-article"> | ||
| <h3>나의 자산</h3> | ||
| <span class="total">159400</span> | ||
| <div class="summary-article-details"> | ||
| <div class="summary-article-details-in"><span>90,600원</span></div> | ||
| <div class="summary-article-details-out"><span>250,000원</span></div> | ||
| </div> | ||
| </article> | ||
| </section> | ||
| */ | ||
| .summary-section{ | ||
| /*display: flex; | ||
| flex-direction: column; | ||
| align-items: center; |
✨ 구현 기능 명세
1. 기본 과제
2. 심화 과제
3. 생각 과제
💎 PR Point
##피드백
🥺 소요 시간, 어려웠던 점
매일 4시간씩🌈 구현 결과물
2023-10-13.3.02.31.mov
웨비의 사진관