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

멋사 3차 과제 - 이예린 #10

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

멋사 3차 과제 - 이예린 #10

wants to merge 3 commits into from

Conversation

dpfls0922
Copy link
Member

@dpfls0922 dpfls0922 commented May 8, 2023

📌 관련 이슈

closed #9

✨ 과제 내용

  • 세미나에서 작성한 페이지에 html, css 파일 보충해서 완성하기

📸 스크린샷(선택)

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

Copy link
Member

@nayujin-dev nayujin-dev left a comment

Choose a reason for hiding this comment

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

card 이외에 어떤 부트스트랩 요소를 추가하신걸까요? 라이브서버 화면캡처 혹은 추가한 부트스트랩에 대한 부분을 언급해주시면 코드리뷰와 과제 확인에 도움이 될 것 같습니다.

수고하셨습니다!
피드백 반영해주시면 확인 후 머지하겠습니다.

<div class="title-box">
<h1>Yerin Lee</h1>
</div>
<div class="card" style="width: 18rem;">
Copy link
Member

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;">
Copy link
Member

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');
Copy link
Member

Choose a reason for hiding this comment

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

웹폰트 임포트 잘하셨네요!

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

Successfully merging this pull request may close these issues.

[FEAT] 멋사 3차 과제🦁
2 participants