-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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(sketch): add sketch package #2720
feat(sketch): add sketch package #2720
Conversation
Color command preview: |
Deploy preview for the-carbon-components ready! Built with commit cc4a9f0 https://deploy-preview-2720--the-carbon-components.netlify.com |
Deploy preview for the-carbon-components ready! Built with commit 27db18e https://deploy-preview-2720--the-carbon-components.netlify.com |
Deploy preview for carbon-components-react ready! Built with commit cc4a9f0 https://deploy-preview-2720--carbon-components-react.netlify.com |
Deploy preview for the-carbon-components ready! Built with commit 61ed5e4 https://deploy-preview-2720--the-carbon-components.netlify.com |
Deploy preview for carbon-components-react failed. Built with commit 61ed5e4 https://app.netlify.com/sites/carbon-components-react/deploys/5ced7e61c3f1b465f64ed389 |
cc @lovemecomputer if you have a chance to review! Going through and adding docs on how the plugin flow works but wanted to point this your way in case you had time before then 😄 |
Plugin download: https://ibm.box.com/s/ag1noz1ueh25rc0f1rlbc2f6klyo10tv |
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.
woo-hoo! Everything worked for me! I didn't check the accuracy of all the values, but I'm assuming they're right since its being generated from code. I can go through and check that though if you want me too.
@aagonzales I know one thing I noticed that was off were the type font weights 😞 for some reason Sketch is thinking we are specifying extra light for plex instead of light (at least on my machine). We might want to just document what are the differences for things like that which will need to be fixed before syncing to a library. I think for colors/themes we should be fine, and icons I think we just will have to wait for someone to tell us when something is busted lol. There are so many and Sketch lags so much 😭 |
@joshblack oh wait the icons all have borders on them |
…2438-generate-sketch-library
…arbon into 2438-generate-sketch-library
Couple of TODOs:
|
@aagonzales here's a link with the latest updates for border issues, let me know if it works! |
@joshblack yup! They're back to normal now. No strokes/borders |
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.
josh gave a walkthrough of the code & plugin, and it seems to be on a pretty good track! it will be good to continue the conversation of ensuring this codebase works for our designers, and the package seems like it has solved the hardest must-have features and is in a good state to merge and keep pushing on it!
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.
🙌 Nice work on this Josh!
Closes #2438
Add
@carbon/sketch
package responsible for creating the Carbon Elements sketch plugin. This plugin provides the following commands as defined by itsmanifest.json
:Notes to reviewers:
README.md
has some details around building and running sketch plugins, alongside project structureprivate
and will not be published as a part of our regular release processChangelog
New
@carbon/sketch
packageChanged
@carbon/icons
16/checkmark--filled
update to correctid
toinner-path
32/checkmark--filled
updatefill-opacity
toopacity
Removed