-
Notifications
You must be signed in to change notification settings - Fork 0
[OD-158] 컬러시스템 구축 & Post, MyPost 합치기 #109
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
Conversation
gustn99
left a comment
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/styles/theme.ts
Outdated
| border: { | ||
| active: '#FF2389', // Pink-500 | ||
| inactive: '#FFBBDA', // Pink-300 | ||
| devider: '#E9E9E9', // Gray-200 |
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.
divider 오타로 보입니다! 수정 부탁드려용
src/styles/theme.ts
Outdated
| secondary: '#4343432', // Gray-800 | ||
| tertiary: '##7B7B7B', // Gray-600 | ||
| caption: '#9E9E9E', // Gray-500 | ||
| onPrimary: '#FFFFFF', // brand.primary 배경 위 텍스트 (White) |
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.
chats 페이지의 matching 탭에서 렌더링되는(아직 구현 안 됨) 매칭 메시지도 흰색으로 설정돼 있습니다! 앞으로의 확장성을 생각했을 때에도 이와 같이 brand.primary 색상 위에서만 흰색 텍스트가 사용된다는 보장이 없기 때문에 onPrimary는 제한된 이름이라는 생각이 듭니다. 따라서 명시적으로 white나, primary 색상과 반대라는 의미에서 primaryContrast, 또는 배경 색상이 있을 때 사용된다는 의미에서 onBackground 같은 방식의 이름은 어떨지 제안드립니다! 꼭 이 이름이 아니더라도 한번 다른 이름을 고려해 보시면 좋을 것 같습니다
| // 여기서부터 기존 fontStyles | ||
| // TODO: 추후 삭제 필요 | ||
| 'body2-light': css` | ||
| font-family: 'Pretendard Variable'; | ||
| font-weight: 300; /* light */ | ||
| font-size: 1rem; | ||
| `, | ||
| 'body4-regular': css` | ||
| font-family: 'Pretendard Variable'; | ||
| font-weight: 400; /* normal */ | ||
| font-size: 0.813rem; | ||
| `, |
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.
이 부분이 남아있어야 사용 중인 기존 폰트가 있는 경우를 검색을 통해 쉽게 찾을 수 있습니다! 전체 폰트스타일 적용이 끝나기 전까지는 해당 코드를 포함하는 게 리팩토링하기에 편리할 것 같아요!
주요 작업 내용
1. 컬러시스템 구축
브랜드 색상
기능별 색상
자주 사용되는 그레이와 핑크 계열 색상은 명도에 따라 50(가장 연한 색상)부터 900(가장 진한 색상)까지 단계별로 정리하여 작성했습니다.
더 필요한 색상 있으시면 말씀해주세요!
2. Post, MyPost 합치기
기타 작업 내용
코드 리뷰 포인트
ex) 브랜드 색상 사용 시 : theme.colors.pink[500] (x), theme.colors.brand.primary(o)
ex) 폰트 색상 지정 시: theme.colors.gray[800] (x), theme.colors.text.secondary(o)
작업 화면