Skip to content
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

주하림 클론코딩 과제 #40

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

주하림 클론코딩 과제 #40

wants to merge 2 commits into from

Conversation

wngkfla01
Copy link

원본 사이트: https://readcereal.com/
과제 사이트: https://snazzy-panda-830a31.netlify.app/

멋진 사진이 가득한 잡지사 사이트를 클론코딩 했습니다.
HTML, CSS로만 구현했습니다.

아쉬운점

  1. 원본 사이트에서는 우측 상단 메뉴 카테고리에 hover를 하면 사진이 바뀌는데, 구현하지 못했습니다.
  2. 사진을 쭉 나열할때 grid형식으로 코드를 조금 더 간결하고 깔끔하게 해보고 싶어요!(요번 과제는 div로 구역을 나눠서 짰습니다.)

index.html Show resolved Hide resolved
index.html Show resolved Hide resolved
</header>
<div class="content">
<div class="inner">
<div class="main">

Choose a reason for hiding this comment

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

원본의 태그는 div가 아닌 main 태그를 사용하였는데요. main 태그를 사용하지 않고 div를 사용한 이유가 있을까요?🤔
** main 태그를 사용하는 이유는 무엇일까요?

Copy link
Author

Choose a reason for hiding this comment

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

원본 구성을 보지 않고 스스로 만들어서 습관적으로 div태그를 사용한 것 같아요. body 요소의 주 콘텐츠를 정의할때 main을 사용하는 것으로 알고 있습니다. 궁금해서 더 찾아봤는데, 문서에서 한번만 사용이 가능하며, article, aside, footer, header, nav 요소의 자손 요소가 되면 안된다고 합니다.

Choose a reason for hiding this comment

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

HTML 리팩토링이나 다른 프로젝트를 진행할 때, 꼭 다양한 시맨틱 태그를 사용해보는 것을 추천드립니다!

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
Copy link

@chanuuuuu chanuuuuu left a comment

Choose a reason for hiding this comment

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

1차 리뷰 완료하였습니다:)

index.html Outdated Show resolved Hide resolved
index.html Show resolved Hide resolved
Copy link

@dmswl2030 dmswl2030 left a comment

Choose a reason for hiding this comment

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

기존에 바뀐부분이 없으면 해당줄에 리뷰를 달수가 없네용ㅠㅠ
370번에 class="button"부분의 클래스 네이밍을 btn으로 수정하면 어떨까요? button이라는 태그명이 있기도하고 "어떤역할_btn" 이런식으로 네이밍을하면 좋을것 같습니다!

Copy link

@DevYBecca DevYBecca left a comment

Choose a reason for hiding this comment

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

리팩토링 시 html 문서는 작업을 안하신 것 같아서 추후에 semantic tag 관련해서 작업해주시면 될 것 같습니다! 고생하셨어요:)

<h5>SUBJECT</h5>
<ul>
<li>
<a href="javascript:void(0)">TRAVEL</a>

Choose a reason for hiding this comment

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

우측 상단 메뉴 카테고리에 hover event가 발생하면 이미지가 바뀌는 효과를 한번 확인해봤습니다! 실제 사이트에서 css 속성으로 작업한 것이 아니라 html의 데이터 속성을 사용하여 background-image를 작업한 것 같더라구요! 저도 생소한 내용이라 서로 공유해보면 좋을 것 같은 포스팅들을 찾아봤습니다!

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