-
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
[Fix] QA 수정사항 반영 및 디자인 시스템 수정사항 반영 #442
Conversation
컬러값 수정됨, blur 추가됨
아이콘 2개 들어가도록 컴포넌트 구조 수정, 여백 미세 조정됨
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -5,6 +5,7 @@ import { css } from '@/styled-system/css'; | |||
import Button from '../Button/Button'; | |||
import Icon from '../Icon'; | |||
|
|||
// 이거 여기에 있는 이유가? | |||
function RightComponentHeader({ rightButtonText = '완료', rightButtonProps, title }: TextButtonHeaderType) { |
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.
마이프로필 레이아웃때문에 만들었었는데 쓰이는 곳없으면 지워도 될듯
원래 의도는 (icon, button 상관없이 component 로 받아서 사용하려고 만드려했는데 사용 하는 곳 없으면 지워도 될 것 같아요
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.
음 말해준 의도랑 다른것 같은데, props 보면 TextButtonHeaderType
예요!
말해준 의도가 맞는지 다시 한번 확인해주실 수 있을까요
🎉 변경 사항