-
Notifications
You must be signed in to change notification settings - Fork 35
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 #74
The head ref may contain hidden characters: "Basic-\uAE40\uC815\uD604-sprint3"
[김정현] Sprint3 #74
Conversation
prettier로 자동정렬시 태그의 꺽쇠 기호가 다음 줄로 내려가는 현상 수정 이미지 이름 직관적으로 수정, 안쓰는 이미지 삭제 img 태그의 alt속성 이미지에 맞게 수정(접근성 향상을 위함)
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.
전반적으로 이미 스프린트2 과제때 스프린트3 내용을 먼저 작업 해주셔서, 그때 받았던 리뷰 사항들을 반영해주신 느낌이네요! 멘토링 & 리뷰 때 말씀드린 내용 하나도 놓치지 않고 반영해주시느라 수고하셨습니다 🙏
.ir_so { | ||
overflow: hidden; | ||
position: absolute; | ||
width: 0; | ||
height: 0; | ||
line-height: 0; | ||
text-indent: -9999px; | ||
} |
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.
멘토링 때 말씀드린 블라인드 텍스트까지 추가해주셨군요! 👍
export function isPasswordHide(btn, input) { | ||
const isPasswordVisible = input.type === 'password'; | ||
input.type = isPasswordVisible ? 'text' : 'password'; | ||
btn.classList.toggle('show', isPasswordVisible); | ||
} |
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.
지난번 리뷰 때 말씀드렸던 내용을 반영해주셨네요! 굳 👍
앞으로도 toggle 을 많이 사용하게될거에요.
@charset "uft-8"; | ||
|
||
/* default */ | ||
* { | ||
box-sizing: border-box; | ||
margin: 0; | ||
padding: 0; | ||
list-style: none; | ||
text-decoration: none; | ||
font-family: 'Pretendard Variable', Pretendard, -apple-system, | ||
BlinkMacSystemFont, system-ui, Roboto, 'Helvetica Neue', 'Segoe UI', | ||
'Apple SD Gothic Neo', 'Noto Sans KR', 'Malgun Gothic', 'Apple Color Emoji', | ||
'Segoe UI Emoji', 'Segoe UI Symbol', sans-serif; | ||
} | ||
|
||
html { | ||
font-size: 16px; | ||
} | ||
|
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 파일 추가 좋습니다! 리뷰반영 굳!
<meta name="description" content="일상의 모든 물건을 거래해보세요" /> | ||
|
||
<meta property="og:title" content="판다마켓" /> | ||
<meta property="og:url" content="https://sprint-fe8-kjh.netlify.app/" /> | ||
<meta | ||
property="og:image" | ||
content="https://sprint-fe8-kjh.netlify.app/assets/images/og_image.jpg" | ||
/> | ||
<meta property="og:description" content="일상의 모든 물건을 거래해보세요" /> |
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.
og 그래프 적용 좋아요!
<link rel="stylesheet" href="../assets/css/register.css" /> | ||
<link rel="stylesheet" href="../assets/css/response.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.
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.
css 파일은 페이지 별로 나누는게 좋을 지 궁금합니다.
-> 질문에 대한 답이 되셨을까요!? common한 성격에 css 파일들과 특정 페이지에서만 사용하는 파일들이 있기 때문에 위 처럼 나누는 방법을 사용한답니다.
html { | ||
font-size: 16px; | ||
} |
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에서 16px 잘해주셨는데요, 그러다보니 rem 사용하실 때 소수점이 나와서 좀 불편하셨을 것 같아요.
이런 방법을 우회하는 방법이 font-size: 62.5% 사용해서 rem 값에 소수점이 발생하지 않도록 하는 방법이 있어요.
링크 남겨드릴테니 다음번에 한번 적용해봐주세요!
https://www.aleksandrhovhannisyan.com/blog/62-5-percent-font-size-trick/
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.
넵 감사합니다!
요구사항
기본
PC: 1200px 이상
Tablet: 768px 이상 ~ 1199px 이하
Mobile: 375px 이상 ~ 767px 이하
375px 미만 사이즈의 디자인은 고려하지 않습니다
랜딩 페이지
로그인, 회원가입 페이지 공통
심화
주요 변경사항
스크린샷
배포 Link입니다!
멘토에게