leejaehyeok97 / introducepage Goto Github PK

안녕하세요! 신나는 2주차 코드리뷰의 시간입니다 😄

지난 피드백에서

• 1. 불필요한 css 속성 정리
• 2. 글자 그림 위로 올리기
• 3. 비율 정렬

모두 잘 체크가 되었네요! 고생하셨습니다 🏆
이번 2주차 과제에서는

1. 다크모드
2. 클릭시 이미지 변환
3. 정보 숨기기
4. 우클릭 시 이벤트(alert) 발생

총 4가지를 구현해주셨군요! 다양한 기능들을 적재적소에 잘 넣어주신 것 같습니다.
onclick외의 다양한 이벤트들에 대해서 시도해보셔도 좋을 거 같아요!
구현에서도 문제 없이 잘 실행이 되네요! 아주 잘하셨습니다 👍

이번주의 피드백입니다!

현재 취미와 잡담 부분에서 사진들이 튀어나오게 보이네요ㅜㅜ

취미 부분에서는 수직 정렬을 함께 사용하는 것이 좋아보여요!

1 2
3 4

위의 형태로 한 번 수정해보시기 바랍니다

잡담 부분에서는 사진의 width 값이 너무 크게 잡혀 있어서 %등으로 수정이 필요해 보입니다!

JS 선언부에서 주석을 달아주셨는데 아주 좋은 습관입니다!
주석을 달아주는 것은 전체적인 코드의 가독성을 높이기에 잘하신 포인트 같습니다!
div 안에 들여쓰기가 되어있지 않은 것들이 있어서 그런 것들도 정리해주시면 훨씬 가독성이 좋은 코드가 될 것 같습니다.

2주차 과제하시는데 정말 고생 많으셨습니다!
시험 기간과 겹쳐서 이것 저것 할 게 많을텐데 아주 잘해주셨어요 👍
궁금하신 사항이 있다면 자주 찾아주세요! 감사합니다!

우선 과제하시느라 고생 많으셨습니다!
margin을 사용하시고 mainbox에 그림자를 약간 추가한 것이 인상적이네요 :D

조금 더 디테일한 웹페이지를 구현하고 싶다면 다음의 리뷰를 한 번 참고해주세요!

1. img class에서 css 속성을 주지 않는 태그에는 불필요한 class를 다 넣을 필요는 없습니다! damwonkia-img, lol-img 두개에만 css 속성이 들어가 있어서 health-img, stock-img 등의 속성이 없는 class는 사용하지 않으셔도 됩니다!

2. 취미div 에 속한 img에서 수직으로 정렬이 되어 있으면 가독성이 떨어지기 때문에 display를 사용하여 수평으로 그림 위에 글자가 갈 수 있도록 정렬하는 것도 좋을 거 같아요! display:flex에 대해서 추가적으로 검색해서 적용해주시면 👍
   

3. 잡담div에 속한 img에서 두 개의 사진을 비율을 맞추어 정렬하시면 더 보기 좋을 것 같습니다!