Skip to content

Conversation

@kwonminjae5700
Copy link

흥미로웠던 점 : HTML CSS 를 이때까지 배웠던 방법대로 따라 치는 것이 아니라 당근 웹 화면을 보고 직접 코드를 짜볼 수 있어서 프런트 엔드에 재미를 가질 수 있게 되었다.
어려웠던 점 : 처음 스스로 코드를 짜봤기 때문에 어떻게 div 로 묶어서 작업해야할지 감이 잘 안 왔으며 javascript 부분이 굉장히 약해서 선배님들이 map 함수를 많이 알려주셨지만 제대로 이해하지 못하고 코드를 작성했다.
궁금한점 : 코드를 작성할 때 CSS 를 효율적으로 작성하지 못 한 것 같아서 CSS 를 효율적으로 쓰는 방법에 대해서 알고 싶고, HTML CSS 를 잘 만들었다 라는 것을 어떻게 판단할 수 있는지 알고 싶다.

@winternuary
Copy link
Contributor

너무 수고많으셨습니다! 어려웠을텐데 질문도 많이 하고 열정적으로 미션에 첨여한게 너무 잘 보여서 기뻤습니다!!
앞으로도 질문 많이많이 하기 약속~~

div로 묶는 부분이 어렵다고 말씀하셨는데 이번 미션하면서 감이오셨을지 모르겠네요!
보통은 섹션별로, 시맨틱 태그를 사용하듯이 최소한으로 사용하는게 좋아요.

<div>
  <p>하하</p>
  <p>호호</p
<div>

이런식으로 필요할 때 딱 한번만 묶어주는게 좋아요

<div>
  <div>
     <p>하하</p>
     <p>호호</p>
  </div>
</div>

이렇게 여러번 묶이게 되면 코드를 읽기에도 힘들고 스타일 지정을 할 때도 굉장히 불편해질거에요.


또한 css를 효율적으로 작성하는 방법에 대해 궁금하다고 하셨는데, css에서 display:flex 사용을 꼭 공부해보셨으면 좋겠어요!
어려울 수 있지만 한번 잘 사용하게되면 굉장히 효율적으로 정렬할 수 있거든요.

  • 그리고 class와 id의 네임 지정에 관해서도 리뷰 남겨 놓았으니 꼭 한번 읽어보면 좋을 것 같습니다.
  • 또한 겹치는 스타일, css 가 똑같다면 하나로 묶는 것도 꼭 기억해주세요!
  • 문단 바꾸기를 할 때 <br>을 사용하는 것 추천하지 않아요. 태그를 나눠서 마진이나 패딩을 주는 것을 추천드려요.

너무 너무 수고많으셨습니다. 리뷰 읽으면서 또 궁금한 점이 생긴다면 언제든 질문 해 주세요!
항상 화이팅입니다~~

Copy link

@anys34 anys34 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다.

Comment on lines +105 to +107
<h4>제휴 문의 광고 문의 PR 문의 IR 문의</h4>
<br>
<h4>이용약관 개인정보처리방침 운영정책 위치기반서비스 이용약관 이용자보호 비전과 계획 청소년보호정책</h4>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

띄워쓰기로 간격을 맞추기 보단 태그로 전체를 묶은 후 display: flex를 사용하는게 좋을 것 같습니다.
그리고
은 지양하는게 좋습니다.
글 참고하면 좋을 것 같습니다.

@kimsiyeon0223
Copy link
Contributor

남다른 열쩡... 정말정말 칭찬합니다.. 덕분에 힘이 많이 나네요!
질문 많고 열정적인 모습이 너무너무 보기 좋아요!!!!! 우리 다음 과제도 열정적이게 해결해봅시다!!
json데이터를 사용할 때 fetch를 사용해야한다는 점을 생각하지 못 해서 과제가 조금 어렵게 나간 것 같아서 미안합니다...
무튼 그래도!!! js파일로 변경하고 잘 해결해 나갔으면 왕칭찬합니다~~
사진 경로에 대한 질문을 해주셨던 것 같은데 이제는 잘 찾을 수 있겠죠???ㅎㅎ
수고많이했어요. 질문이 많아도 괜찮고 늦은 시간이여도 괜찮아요. 편하게 연락해주세용!

<div class="head_chat">
<h4>채팅하기</h4>
</div>
<input type="text" class="head_search" placeholder="물품이나 상품을 검색해보세요">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

<input type="text" class="head_search" placeholder="물품이나 상품을 검색해보세요" />

태그가 닫히지 않았습니다.. 위와같이 셀프 클로징 태그를 써주세요!!

<p class="fleamarket_p">동네 주민들과 가깝고 따뜻한 거래를<br>지금 경험해보세요.</p>
</div>
<div class="fleamarket_picture">
<img src="../assets/fleamarketpicture.jpg" alt="플리마켓 사진" width="400" height="315">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

인라인 스타일은 지양해주세요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants