Skip to content
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

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

jinjoo-jung
Copy link

@jinjoo-jung jinjoo-jung commented Jul 28, 2023

SSWU 홈페이지 클론 코딩

구현한 내용

  • html, css 를 사용하여 구현
  • 시맨틱 태그를 최대한 사용
  • Flex를 사용하여 화면을 배치
  • netlify를 이용하여 사이트 배포

아쉬웠던 점

  • 메뉴바를 클론 사이트와 동일하게 구현하지 못한 점
  • 오른쪽 사이트 플로팅과, 슬라이드를 구현하지 못한 점
  • 시맨틱 태그를 이용하였지만 태그를 깨끗하게 사용하지 못 한 느낌이 들었다

참고 이미지

img1

Select 태그 이용

스크린샷 2023-07-28 오후 11 31 43

hover를 통한 색 바꿈

vedio1.mov

Develop

HTML CSS JavaScriptVisual Studio CodeGithub

@netlify
Copy link

netlify bot commented Jul 28, 2023

Deploy Preview for mellifluous-gnome-003496 ready!

Name Link
🔨 Latest commit 0990e88
🔍 Latest deploy log https://app.netlify.com/sites/mellifluous-gnome-003496/deploys/64c3d700eaf1e600073960da
😎 Deploy Preview https://deploy-preview-56--mellifluous-gnome-003496.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@wkdtnqls0506
Copy link

시멘틱 태그를 되게 잘 활용하신거 같아요!👍👍
저는 nabvar에 hover시 또 다른 navbar가 보이게 설정하는 부분에서 되게 시간을 많이 잡아먹었었는데 진주님 코드 보고 더 많은 걸 배워갑니닷!😁😀

@minseokiim
Copy link

고생하셨어요 👍 리드미가 깔끔해서 보기 좋아요~
파비콘만 추가해주시면 더 좋을것 같습니다!

@jongsujin
Copy link

고생하셨습니다! cursor 부분을 미처 생각하지 못했는데 진주님 코드보고 배워갑니다! 리드미도 자세하게 설명해주셔서 작성법도 많이 배웠습니다!

@syb0127
Copy link

syb0127 commented Aug 4, 2023

아이콘이랑 버튼이 많이 보여서 위치 잡는게 오래 걸리셨을거 같은데 정말 고생 많으셨습니다. 시맨틱 태그에 대해서도 많이 배워가요~

Copy link

@jungHyeonS jungHyeonS left a 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;">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

인라인 스타일은 코드의 가독성을 떨어트려서 최대한 css 파일에 작성해주시면 좋습니다~

Comment on lines +18 to +24
<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>

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">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

추후 작업하실때는 사용하신 img 태그에 alt가 빠지진 않았는지 점검하시면 좋을꺼같습니다~

Comment on lines +67 to +71
.dropdown {
display: inline-block;
position: relative;

}

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를 넣어주시면 더 간단하게 작성가능할꺼같습니다~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants