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

KDT5_LeeYongSoo #60

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

KDT5_LeeYongSoo #60

wants to merge 4 commits into from

Conversation

dragon-it
Copy link

@dragon-it dragon-it commented Apr 5, 2023

사이트 클론

레퍼런스 사이트 : https://www.pulmuone.co.kr/pulmuone/main/Index.do#anchor1
클론 코딩 사이트 : https://rainbow-salmiakki-81ec15.netlify.app

완성도

50%도 되지 않지만 기본적으로 헤더, 푸터, 사이트 연결 등 공통적으로 포함되어 있는 부분은 연결 해놓았습니다.

설명

헤더 부분의 풀드랍 부분과 서브 메뉴바에만 간단한 자바스크립트를 사용하였습니다.

클론 한 사이트가 아닌 원본에 연결된 링크들은
a 링크에 target _blank를 주었습니다.
때문에 새탭으로 나오는 웹페이지는 클론 사이트가 아닙니다.

원본 사이트의 마우스 스크롤 위치에 따라 컨텐츠가 아래에서 나오게 하는 자바스크립트 동작은
키프레임으로 컨텐츠 마다 지연 시간을 늘려서 마우스 스크롤에 따라 컨텐츠가 나오는 것 처럼
느껴지게 만들었습니다.

메인 페이지에서 section을 만들고 앵커를 연결해서 이동할 수 있게 했습니다.

@dragon-it dragon-it changed the title 풀무원 클론 코딩 KDT5_LeeYongSoo Apr 6, 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 잘 작성해 주신 것 같습니다.
다른 페이지를 더 만드시는 것 보단 다른 기능
혹은 다른 구성의 페이지를 시도해 보시거나
scss 적용 등을 고려해보시면 좋을 것 같습니다.

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

HTML

시멘틱 하게 잘 작성해 주신 것 같아요.
다만 너무 중복되는 내용의 많은 페이지를 만들어 주신 것 같아요.
동일하거나 거의 비슷한 구성의 페이지를 여러가지 하는 것은
큰 의미는 없지 않을까 생각이 드는 것 같습니다.

CSS

css 잘 작성해 주신 것 같습니다.
BEM 방법론도 잘 고려해 주신 것 같아요.
scss활용도 고려해 보시면 더욱 좋을 것 같습니다.

JS

필요한 부분에 대한 동작을 간결하게 잘 작성하신 것 같습니다.

CI/CI.html Outdated
</div>
</div>
<div>
<img src="../images/sub_vis04_01.jpg" alt="">
Copy link
Member

Choose a reason for hiding this comment

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

어떤 이미지인지 alt 에 입력 해주면 좋을 것 같습니다.

Copy link
Author

@dragon-it dragon-it Apr 10, 2023

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.

2 participants