Conversation
Creat: clone_sungshin_potal
update: sungshin_potal
Update: SungShinPotal_footer
Kdto jung jin joo
…o KDTO_JungJinJoo
✅ Deploy Preview for mellifluous-gnome-003496 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
시멘틱 태그를 되게 잘 활용하신거 같아요!👍👍 |
|
고생하셨어요 👍 리드미가 깔끔해서 보기 좋아요~ |
|
고생하셨습니다! cursor 부분을 미처 생각하지 못했는데 진주님 코드보고 배워갑니다! 리드미도 자세하게 설명해주셔서 작성법도 많이 배웠습니다! |
|
아이콘이랑 버튼이 많이 보여서 위치 잡는게 오래 걸리셨을거 같은데 정말 고생 많으셨습니다. 시맨틱 태그에 대해서도 많이 배워가요~ |
jungHyeonS
left a comment
jungHyeonS
left a comment
There was a problem hiding this comment.
진주님 작업하시느라 고생하셨습니다~
html 구조도 깔끔하게 작성해주셨고 css도 깔끔하게 작성해주신거 같습니다
리뷰 드린 부분만 추후 작업할때 참고해주시면 좋을꺼같습니다~
|
|
||
| </head> | ||
|
|
||
| <body style="margin: 0;"> |
There was a problem hiding this comment.
인라인 스타일은 코드의 가독성을 떨어트려서 최대한 css 파일에 작성해주시면 좋습니다~
| <div class="LOGIN">LOGIN</div> | ||
| <img class="SEARCH_img" src="/sungshin_img/icon-search.jpg"> | ||
| <div class="SEARCH">SEARCH</div> | ||
| <div class="Korean">Korean</div> | ||
| <img class="Arrow" src="/sungshin_img/arrow-location.jpg"> | ||
| <div class="SSS">수강신청시스템</div> | ||
| <div class="Potal">성신포탈</div> |
There was a problem hiding this comment.
이부분은 코드 스타일에 관련한 부분인데 일반적으로는 class 명을 대문자로 시작하지는 않습니다! id 같은경우는 대문자로 시작하는경우가 있기는한데 클래스는 거의 소문자로 작성하는 부분 참고 부탁드립니다~
| <div class="topic-5">[조선일보] 성신여대 조소과, 태안 캐릭터 모래조각 페스타 대상 수상</div> | ||
| </div> | ||
| <div class="section-6-div2"> | ||
| <img class="vedio" src="/sungshin_img/img-media.jpg"> |
There was a problem hiding this comment.
추후 작업하실때는 사용하신 img 태그에 alt가 빠지진 않았는지 점검하시면 좋을꺼같습니다~
| .dropdown { | ||
| display: inline-block; | ||
| position: relative; | ||
|
|
||
| } |
There was a problem hiding this comment.
코드를 확인해보니 dropdown 부분을 Inline-block 으로 잡아주신 부분확인하였습니다! 해당부분은 추후 반응형 까지 고려하면 nav 클래스에 display:flex를 넣어주시면 더 간단하게 작성가능할꺼같습니다~
6 participants
SSWU 홈페이지 클론 코딩
구현한 내용
아쉬웠던 점
참고 이미지
Select 태그 이용
hover를 통한 색 바꿈
vedio1.mov
Develop