-
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
feat: color scheme에 따른 팔레트 색상 분기 #170
base: develop
Are you sure you want to change the base?
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.
기존 시스템에 변경점이 생겨서 실 서비스에 영향이 없으려면 apps
하위의 각 서비스에 요 파일 import
를 추가해야할 것 같아요!
import '@boolti/ui/src/theme.css
흠... 뭔가 임포트가 안이쁜데 @boolti/ui/theme.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.
exports map을 사용하면 import 경로를 이쁘게 바꿀 수 있는데, boolti/ui에 별도의 빌드 과정이 없어서 어떻게 번들링되고 있는지 모르겠어서 지금처럼 해두었어요.
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.
생각해보니 당장 병합 가능한 상태가 아니긴 하네요. 일단 draft로 변경해둘게요.
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.
https://jsramblings.com/using-yarn-symlinks-to-share-code-between-apps/amp/
packages
하위에 있는 패키지들은 yarn이 symlink를 이용해서 번들링할때 직접 가져와서 같이 번들 되고 있는 것 같네요~!
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.
exports map 추가해 두었어요 9de60d1
다크모드 색상값을 적당히 적용해 두었어요. HSLuv 색공간을 이용해서 lightness만 반전했어요. 그대로 쓰기는 좀 어렵고, 여기서 수정해나간다는 느낌으로 보면 좋을 것 같아요.
|
prefers-color-scheme은 OS에서 컬러 모드를 변경하거나, 크롬 개발자 도구에서 변경할 수 있습니다.