-
Notifications
You must be signed in to change notification settings - Fork 1
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
[#12] 정산페이지 레이아웃 작업 및 더미 데이터를 사용한 기능 구현 #55
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.
반응형도 엄청 빨리 하셨네요..! 정산 페이지도 까다로웠을 거 같은데 잘 해주신 거 같아요...!! 수고하셨습니다 👍🏻
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에라도 반영해주심 좋을 것 같습니다!
@media (max-width: 900px) { | ||
margin: 10px 20px; | ||
|
||
display: flex; | ||
flex-direction: column; | ||
justify-content: center; | ||
} | ||
`; |
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.
미디어 쿼리 쓰는 부분 현재 theme파일 들어가면, 상수화되어 있는데 사용하시는게 좋을 것 같습니다!
추후 변경될 수 있는 너비라 생각해서 다음 PR에서라도 반영되면 좋을 것 같아요!
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에서는 반영해놓겠습니다 !
@media (max-width: 500px) { | ||
width: 100%; | ||
} |
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.
이런 (500px) 부분도, 지훈님이 설정해주신 theme파일 지오님 영역에서 상수화해서 관리하면 좋을 것 같습니다!
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.
넵 알겠습니다 !!
close #12
Description
추가된 라이브러리는 다음과 같습니다.
유의할 점 및 ETC (Optional)
스크린샷 (Optional)
-.Clipchamp.mp4