-
Notifications
You must be signed in to change notification settings - Fork 3
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
Valet Theme Updater — Token Transformer #83
Conversation
✅ Deploy Preview for vip-design-system-components ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
d11991b
to
4c79d2a
Compare
4c79d2a
to
7100190
Compare
@@ -56,7 +56,7 @@ const outline = { | |||
outlineStyle: 'solid', | |||
outlineColor: '#ffffff', | |||
outlineWidth: '1px', | |||
boxShadow: `0 0 0 1px #fff, 0 0 0 3px ${ getColor( 'focus' ) }`, | |||
boxShadow: `0 0 0 1px ${ getColor( 'focus', 'inset' ) }, 0 0 0 3px ${ getColor( 'focus' ) }`, |
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.
This is awesome 💯
LGTM |
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.
Amazing!
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.
Awesome! It's cool that you can reuse color references in the design system and can drop in the tokens as-is.
Are you planning to integrate other tokens like typography in here from the design system as well?
Yes, that's the idea for future improvements @alecgeatches 🥳 |
Description
Introducing a way to have the VIP Design System (Figma files) integrated with all our React components 🥳 . The long explanation is described in the README.md file changes in this Pull Request.
The long history short is: We now can read all theming information we need from a tokens JSON file and parse it so the React theme can be read and used in the components system.
New npm script command added:
Checklist
Steps to Test
npm run theme-update
.src/generated/valet-theme.json
filenpm run dev