-
Notifications
You must be signed in to change notification settings - Fork 0
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
멋사 3차 과제 - 이예린 #10
base: main
Are you sure you want to change the base?
멋사 3차 과제 - 이예린 #10
Conversation
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.
card 이외에 어떤 부트스트랩 요소를 추가하신걸까요? 라이브서버 화면캡처 혹은 추가한 부트스트랩에 대한 부분을 언급해주시면 코드리뷰와 과제 확인에 도움이 될 것 같습니다.
수고하셨습니다!
피드백 반영해주시면 확인 후 머지하겠습니다.
assignment/3th_index.html
Outdated
<div class="title-box"> | ||
<h1>Yerin Lee</h1> | ||
</div> | ||
<div class="card" style="width: 18rem;"> |
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.
여기서부터 쓴 card 부트스트랩은 수업때 한 내용인것같네요! 커밋 메세지와 수정된 코드 내용이 일치하게끔 작성하시면 코드리뷰가 더 도움이 될것같아요.
<h1>Yerin Lee</h1> | ||
</div> | ||
<section> | ||
<div class="card" style="width: 35rem;"> |
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.
같은 내용이 이전 커밋에도 있는것 같습니다.
혹시 다른 내용인데 제가 놓친거라면 말씀해주세요!
아니면 같은 내용을 다시 커밋한 이유가 따로 있는걸까요?
커밋 메시지를 더 구체적으로 작성하거나, 혹은 PR을 날릴때 본문에 그에대해 언급해주시면 더 원활한 코드리뷰가 가능해요.
@@ -0,0 +1,44 @@ | |||
@import url('https://fonts.googleapis.com/css2?family=IBM+Plex+Sans+KR:wght@300;400;600&display=swap'); |
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.
웹폰트 임포트 잘하셨네요!
📌 관련 이슈
closed #9
✨ 과제 내용
📸 스크린샷(선택)
📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들