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

첫번째 클론코딩 과제 #78

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

첫번째 클론코딩 과제 #78

wants to merge 2 commits into from

Conversation

dosick0
Copy link

@dosick0 dosick0 commented Apr 11, 2023

첫번째 클론코딩 과제.
부족하지만 최대한 구연가능성 높은 사이트로 작업했습니다.

Copy link
Member

@iamidlek iamidlek left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.
잘 구현 하여주셨고 시멘틱하게 작성하시고
테이블 태그도 잘 사용 하신 것 같아요.

중복되는 부분은 코멘트 하지 않았습니다.

HTML

시멘틱 하게 작성하려다 보니 불필요하게 감싼 태그도 있는 것 같습니다.
그리고 렌더링된 화면에 영향을 주는 오타도 포함되어있어
그런 부분을 조금 더 신경 쓰면 좋을 것 같습니다.

CSS

BEM 방법론이 요구 사항에 있었으므로
조금 더 고려해서 작성해 보시면 좋을 것 같습니다.

<link rel="stylesheet" href="main.css">
</head>
<body>
<section id="number_1">
Copy link
Member

Choose a reason for hiding this comment

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

전체 내용을 section으로 감쌀 필요가 있을까요?
class가 아닌 id 를 주신 이유도 궁금합니다.

/* gitpadding-bottom: 100px; */
background: url(https://www.kovo.co.kr/images/intro/bg_pointro_2223_2.jpg) center top no-repeat;
}
.frist {
Copy link
Member

Choose a reason for hiding this comment

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

코드 작성 및 변경 후 저장 할 때
코드 자동 정렬되는 기능을 찾아보시면 좋을 것 같습니다.

</footer>
</section>
</body>
</html>s
Copy link
Member

Choose a reason for hiding this comment

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

s 뒤에 붙은 s는 불필요 할 것 같습니다. 실제 렌더링 시 영향을 주는 부분이라 주의하면 좋을 것 같습니다.

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.

2 participants