365kim / web_study Goto Github PK

https://codepen.io/365kim/pen/yLONJaz

역시 열심히 해주셔서 너무 감사합니다 ㅠㅠ

간단하게 사이트 보고 피드백 드립니다~

• 좌측 프로필 이미지가 .container 를 벗어나서 .sticky 영역을 침범하고 있습니다.

일부러 그렇게 스타일링을 하셨다면 맞지만 혹시 벗어난거라면 이미지에 width: 100%를 줘서 .left 가로길이에 맞추거나 .sticky영역에 z-index: 1000 정도? 주시면 .sticky영역에 이미지가 침범하지 않습니다!

• .container클래스를 중간과 하단 두개를 사용하고있습니다.

하단은 명칭에 맞게 footer, 중단은 container보단 content? main? 이런 정확한 명칭이 좋지 않을까.. 합니다! container는 div같은 공간의 묶음이라 표현하는것 같아서..
HTML5인 시멘틱태그로 명칭을 확실하게 해주시면 더욱 보기좋은 코드가 될것같습니다!

<div id="footer" class="container"> ... </div>
<div id="footer"><div class="container>...</div></div>
<footer class="container">...</div>

html5 시멘틱태그 => https://snusang.tistory.com/4

• .right 클래스 안에 div공간을 이렇게 나누는건 어떨까요?

각 section마다 아래와 같이 구분하면 반응형 작업을 할때도 공간구분이 되어있어 좀더 쉬운것 같습니다.
(이건 개인 퍼블리싱 하는것에 다른것같습니다 ㅎㅎ.. )

<div id="pinned" />...</div>
<div id="contribution-in-last-year">...</div>
<div id="contribution-activity">...</div>
또는
<section id="pinned" />...</section>
<section id="contribution-in-last-year">...</section>
<section id="contribution-activity">...</section>

정말 열심히 해주셔서 너무 감사합니다 ㅠㅠ
시간날때 열심히 피드백 해드리겠습니다!

모르시는것 있으면 따로 디코에 말씀하셔두 됩니다! 같이 화면보면서 공부해도 되요!!
감사합니다~