-
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
[TASK-49] style: 디자인 시스템 변경사항 반영 (국문/영문 폰트 설정, 공용 컴포넌트 기능 추가) #12
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.
늦은시간에 리뷰드려 죄송합니다!
전체적인 부분은 크게 수정할 부분이 없어보입니다! 코멘트 확인부탁드립니다!
@@ -22,7 +22,7 @@ const EmailInput = ({ mode }: EmailInputProps) => { | |||
setEmail(''); | |||
}; | |||
|
|||
const checkEmailStatus = (email: string) => { |
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.
혹시 이 부분은 제가 수정해도 괜찮을까요? validate 쪽이 많이 바뀔 것 같아서 제가 한 번에 수정하겠습니다!
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.
아항 네 좋습니다 :)
src/components/Navbar/index.tsx
Outdated
<div className='rounded-full bg-white w-8 h-8 flex-shrink-0'></div> | ||
<button | ||
className='button-m bg-main px-6 py-2 rounded-full text-white flex-shrink-0' | ||
style={{ whiteSpace: 'nowrap' }} |
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.
여기만 인라인 스타일 적용하신 이유가 있을까요? whitespace-nowrap
으로는 적용이 안되나요..?
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.
Layout 스타일 적용이 안 되는 부분이 있어서 이것저것 style
로 시도하다가 미처 삭제하는 걸 잊었네요 ㅠㅠ whitespace-nowrap
없어도 디자인한 UI대로 나와서 제거해서 push 했습니다!
📝 작업 내용
className
에font-sans
CSS 클래스를 작성하면 적용되도록 추가FilterDropdown
컴포넌트3-1) 아이콘 변경 (Dropup/Dropdown)
3-2) 선택된 옵션 강조 기능 추가
Pagination
컴포넌트4-1) MoreHorizontal 아이콘 추가
4-2) 마지막 페이지 표시
4-3) 페이지 이동 시 버튼 위치 고정 및 페이지 그룹 전환 로직 구현
📸 스크린샷 (선택)
💬 전달사항