김준희 토스(toss) 메인 페이지 클론 코딩

[dev-junehee]

안녕하세요. FE 부트캠프 5기 김준희입니다.
개인 프로젝트 과제 풀 리퀘스트 제출합니다!

• 원본 홈페이지 : https://toss.im/
• 클론 코딩 과제물 : https://funny-vacherin-5977e2.netlify.app/

자세한 내용은 README.md 내에 작성해두었습니다.
감사합니다.

[hbsongk]

commit Header 부분에 내용들을 종합적으로 요약해주는 정보가 들어가면 좋을 것 같아요. ex) build header frame

[dev-junehee]

commit Header 부분에 내용들을 종합적으로 요약해주는 정보가 들어가면 좋을 것 같아요. ex) build header frame

오 홍빈님! 감사합니다.
커밋 메세지 순서가 중요한줄 알았는데 다음엔 의미있는 내용으로 채워서 보기 쉽게 적용해보겠습니다!

[dev-junehee]

�창의 크기를 늘렸다 줄였을 때 늘 가운데 정렬을 하기 위해서

position: relative;
display: flex;
justify-contents: center;
align-items: center

position: absolte;
top:
.
.
.

위 속성을 많이 이용했는데, 아이템 요소 위치값에 늘 마이너스 값이 자주 사용되었습니다..!
물론 배치를 위해 마이너스 값을 넣을 수는 있지만 다른 효율적인 방법이 없을지 궁금합니다.
항상 마이너스 값으로 배치하다보니 잘못하고 있는 것 같다(?)는 생각이 들었습니다 🧐

[happyhermann]

flex/positon 사용하신 것 좋습니다!

스크롤 애니메이션을 통해 보여지고 안 보여지고를 js로 동적인 UI 구성한 것에 대해 칭찬하고 싶습니다!

BEM은 코딩하다 보면 익숙해질 것이니 괜찮을거 같고요

메타태그 잘 사용하셨고
다만 div보다 nav를 리스트 최상단에 배치했으면 더 좋았을듯 싶네요!

또 h1 태그가 존재하지 않는데 h1태그는 구글 웹크롤링 / SEO에서 매우 중요한 요소입니다! 그러므로 한 페이지내에 h1태그는 1개만 있어야하며, 홈페이지내에 1개 이상 반드시 존재해야합니다 앞으로 유의하시면 좋을것 같네요!
또한 헤딩 태그는 순서대로 써야합니다 h4다음에 h2가 오면 시멘틱하다고 할 수 는 없겠네요
br은 닫은 태그로 쓰시면 더 좋을거 같고요
li태그는 ul태그안에 쓸 순 있지만 ul태그가 li태그안에 있으면 시맨틱하지 않게 볼 수 있습니다!
nav > ul > li 순서를 유의하시면 좋을거 같네요

header/ footer 나눠서 한 것은 잘 하셨습니다

첫 프로젝트인데 잘 하신거 같네요
고생하셨습니다

[dev-junehee]

flex/positon 사용하신 것 좋습니다!

스크롤 애니메이션을 통해 보여지고 안 보여지고를 js로 동적인 UI 구성한 것에 대해 칭찬하고 싶습니다!

BEM은 코딩하다 보면 익숙해질 것이니 괜찮을거 같고요

메타태그 잘 사용하셨고 다만 div보다 nav를 리스트 최상단에 배치했으면 더 좋았을듯 싶네요!

또 h1 태그가 존재하지 않는데 h1태그는 구글 웹크롤링 / SEO에서 매우 중요한 요소입니다! 그러므로 한 페이지내에 h1태그는 1개만 있어야하며, 홈페이지내에 1개 이상 반드시 존재해야합니다 앞으로 유의하시면 좋을것 같네요! 또한 헤딩 태그는 순서대로 써야합니다 h4다음에 h2가 오면 시멘틱하다고 할 수 는 없겠네요 br은 닫은 태그로 쓰시면 더 좋을거 같고요 li태그는 ul태그안에 쓸 순 있지만 ul태그가 li태그안에 있으면 시맨틱하지 않게 볼 수 있습니다! nav > ul > li 순서를 유의하시면 좋을거 같네요

header/ footer 나눠서 한 것은 잘 하셨습니다

첫 프로젝트인데 잘 하신거 같네요 고생하셨습니다

h1태그의 중요성을 알고는 있었는데 하다보니 완전 잊어버렸습니다.
다음 과제부터 다시 잘 적용해보겠습니다! 감사합니다 :)

[ChoEun-Sang]

JS 활용해서 애니메이션 효과 너무 잘 하신 거 같습니다~!