KDT5_YangJunYong

[azure0929]

innisfree 메인 페이지 클론코딩

1.배포 결과물
https://innisfree-clone-coding.netlify.app/

2.원본 사이트

https://www.innisfree.com/kr/ko/Main.do

원본 사이트에서 개발자 도구로 class와 css를 보고 구현했습니다.
javascript는

-swiper : event slide 부분

-scrollTop : aside 버튼 클릭 시 Top으로

-scroll show & hide : 스크롤 시 header 영역을 벗어나면 fix-head를 show & hide로 구현

html과 css를 개발자 도구를 보면서 작성했지만 
slide와 hover 부분을 구현하는 부분에 있어 어려움을 겪었습니다.
아래와 같은 css 부분을 중점적으로 리뷰 부탁드립니다.

1).eventbn section의 전반적인 css
2).shopforu .shopforu-list ul
3).clientReview .pdtList .pdtCont:hover 시
.name2의 내용이 overflow: visible 이외에 좀 더 효율적인 방법

[iamidlek]

고생하셨습니다.
html, css 모두 요구 사항에 맞게 잘 작성해 주신 것 같습니다.

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

HTML

시멘틱하게 작성해 주시려고 고려한 사항이 많이 있으신 것 같아서
좋은 것 같습니다.

CSS

BEM 방법론이 적용된 부분이 많지는 않은 것 같아
더 적극적으로 적용해 보는 것도 좋을 것 같습니다.
중복되는 속성 적용이 다수 있는 것 같아
scss 활용해 보시는 것도 좋을 것 같습니다.

JS

jquery 없이도 충분히 구현 가능한 내용이어서
바닐라 자바스크립트 상태로 구현해 보시는 것도 좋을 것 같습니다.

[iamidlek]

태그의 기본 값은 따로 적어주지 않으셔도 됩니다.

[iamidlek]

important 주신 이유가 있을까요?