-
Notifications
You must be signed in to change notification settings - Fork 26
[유경민] sprint3 #83
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
[유경민] sprint3 #83
The head ref may contain hidden characters: "Basic-\uC720\uACBD\uBBFC-sprint3"
Conversation
|
스프리트 미션 하시느라 수고 많으셨어요. |
화면을 줄였을 때 뚝뚝 끊기는 느낌이 조금 심한데 해결 방법이 있을까요?오잉? 배포 화면으로 봤을 때는 큰 문제가 없어보여요. |
commit 단위를 더욱 자주, 작게 해보시는건 어떠실까요?git을 다룰 때 commit은 "언제 해야 하는가"를 생각해보신 적 있으신가요?
그럼 커밋을 언제 해야 할까요?저는 다음과 같은 룰을 지키며 커밋을 하는걸 권장 드립니다:
관련하여 읽으시면 좋은 아티클을 추천드릴게요:tl;dr관련 변경 사항 커밋 자주 커밋 미완성 작업을 커밋하지 마십시오 커밋하기 전에 코드를 테스트하세요 |
커밋 타입도 한 번 고려해보세요 !tl;dr:커밋 메시지 형식 type: Subject
body
footer기본적으로 3가지 영역(제목, 본문, 꼬리말)으로 나누어졌다. 메시지 type은 아래와 같이 분류된다. 아래와 같이 소문자로 작성한다. |
| .container-text p#top br, | ||
| h3 br{ | ||
| display: none; | ||
| } |
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.
혹시 아직 프리티어가 적용이 잘 안되었을까요?
프리티어는 설치하신 후 포매팅을 작동 하셔야합니다. 😉
| .container-text p#top br, | |
| h3 br{ | |
| display: none; | |
| } | |
| .container-text p#top br, h3 br{ | |
| display: none; | |
| } |
사소한 줄바꿈, 띄어쓰기 등 코드를 작성하시다보면 자연스럽게 불규칙해지는 경우가 많아요.
이를 돕기 위해서 개발 커뮤니티에서는 개발 경험 향상을 위한 여러가지 도구들이 있는데요.
보편적으로 많이 사용되는 툴은 prettier입니다 !
Prettier: https://prettier.io/
prettier는 vscode plugin에서 install하고 실행할 수 있습니다 ! 🤗
macos 경우
option+shift+f
windows의 경우alt+shift+f
혹시 적용하시기 어려움이 있다면 편하게 DM 주세요 😉😉
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.
굿굿 ! reset.css를 작성하셨군요 !
이제 경민님의 프로덕트는 어떤 브라우저로 봐도 일관된 UI를 보여줄 수 있겠어요 👍👍
| <link rel="icon" href="images/favicon-panda.png" type="image/x-icon" /> | ||
| <meta property="og:title" content="판다마켓" /> | ||
| <meta property="og:description" content="일상의 모든 물건을 거래해보세요" /> | ||
| <meta property="og:image" content="https://file2.nocutnews.co.kr/newsroom/image/2019/01/18/20190118151430114340_0_800_451.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.
ㅋㅋㅋㅋ 경로가 이상해서 봤더니 판다 사진이었군요 🤣
| <link rel="icon" href="images/favicon-panda.png" type="image/x-icon" /> | ||
| <meta property="og:title" content="판다마켓" /> | ||
| <meta property="og:description" content="일상의 모든 물건을 거래해보세요" /> | ||
| <meta property="og:image" content="https://file2.nocutnews.co.kr/newsroom/image/2019/01/18/20190118151430114340_0_800_451.jpg" /> | ||
| <meta property="og:url" content="https://pandapandapanda.netlify.app/" /> | ||
| <meta property="og:type" content="website" /> | ||
| <link rel="stylesheet" href="css/reset.css" /> |
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.
굿굿 ! 메타 태그를 올바르게 적용하셨네요 👍👍
| <picture class="logo-picture"> | ||
| <source srcset="images/logo-typo.svg" media="(max-width: 767px)"> | ||
| <img src="images/logo.png" alt="로고" class="로고" /> | ||
| </picture> |
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만으로 반응형을 잘 구현하셨네요 ㄷㄷㄷ 👍👍
| } | ||
|
|
||
| /*tablet*/ | ||
| @media (max-width: 1199px){ |
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.
굿굿 ~! 미디어 쿼리 적용 잘 하셨네요 👍👍
| </div> | ||
| </div> | ||
| </section> | ||
| <section> |
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.
굿굿 ! 각 컨텐츠를 section으로 구분하셨네요 👍
더욱 더 의미있는 문서로 거듭나는게 느껴집니다 👍👍
|
수고하셨습니다 경민님 ! 멋진 시도 응원합니다! 💪💪💪 |
✅기본
✅공통
✅로그인 / 회원가입 페이지 공통
✅ 심화 요구사항
/) 공유 시 좌측 예시와 같은 미리보기를 볼 수 있도록 메타 태그를 설정함배포 사이트
https://pandapandapanda.netlify.app/
스크린샷
멘토에게
화면을 줄였을 때 뚝뚝 끊기는 느낌이 조금 심한데 해결 방법이 있을까요?
미션 2 리뷰 내용은 우선 일부만 반영했습니다ㅠ..ㅠ 3 리뷰 받은 후에 같이 수정하겠습니다!
잘부탁드립니다 🙇♂️🙇♂️🙇♂️