-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
KDTO_JungJinJoo SSWU 홈페이지 클론코딩 #56
base: main
Are you sure you want to change the base?
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 부분을 미처 생각하지 못했는데 진주님 코드보고 배워갑니다! 리드미도 자세하게 설명해주셔서 작성법도 많이 배웠습니다! |
아이콘이랑 버튼이 많이 보여서 위치 잡는게 오래 걸리셨을거 같은데 정말 고생 많으셨습니다. 시맨틱 태그에 대해서도 많이 배워가요~ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
진주님 작업하시느라 고생하셨습니다~
html 구조도 깔끔하게 작성해주셨고 css도 깔끔하게 작성해주신거 같습니다
리뷰 드린 부분만 추후 작업할때 참고해주시면 좋을꺼같습니다~
|
||
</head> | ||
|
||
<body style="margin: 0;"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
인라인 스타일은 코드의 가독성을 떨어트려서 최대한 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이부분은 코드 스타일에 관련한 부분인데 일반적으로는 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
추후 작업하실때는 사용하신 img 태그에 alt가 빠지진 않았는지 점검하시면 좋을꺼같습니다~
.dropdown { | ||
display: inline-block; | ||
position: relative; | ||
|
||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코드를 확인해보니 dropdown 부분을 Inline-block 으로 잡아주신 부분확인하였습니다! 해당부분은 추후 반응형 까지 고려하면 nav 클래스에 display:flex를 넣어주시면 더 간단하게 작성가능할꺼같습니다~
SSWU 홈페이지 클론 코딩
구현한 내용
아쉬웠던 점
참고 이미지
Select 태그 이용
hover를 통한 색 바꿈
vedio1.mov
Develop