Skip to content
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

Adding all color modes to figma export #805

Merged
merged 9 commits into from
Feb 20, 2024

Conversation

lukasoppermann
Copy link
Contributor

This PR adds transformations to be able to import all colors modes into Figma.

@lukasoppermann lukasoppermann requested a review from a team as a code owner February 16, 2024 14:26
@lukasoppermann lukasoppermann requested review from a team, rezrah and pksjce February 16, 2024 14:26
Copy link

changeset-bot bot commented Feb 16, 2024

🦋 Changeset detected

Latest commit: 882384a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/primitives Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

github-actions bot commented Feb 16, 2024

Variables changed
No variables changed

@@ -30,7 +38,7 @@ export const buildFigma = (buildOptions: ConfigGeneratorOptions): void => {
}
//
for (const {filename, source, include} of themes) {
if (['light', 'dark'].includes(filename)) {
if (!['light-high-contrast', 'dark-high-contrast', 'dark-dimmed'].includes(filename)) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about color blind themes?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just disabling them to test if they are problematic, they are. It is just a temp step.

@lukasoppermann
Copy link
Contributor Author

So it works great for all modes that have no specific base scale, so everything but light-high-contrast, dark-high-contrast, dark-dimmed.

So I need to find a fix for those three.

Copy link
Contributor

Design Token Diff

No design tokens changed

@lukasoppermann lukasoppermann force-pushed the @lukasoppermann/add-figma-colormodes branch from 79e6f0b to 1458cde Compare February 19, 2024 13:00
@github-actions github-actions bot temporarily deployed to storybook-preview February 19, 2024 13:01 Inactive
@github-actions github-actions bot temporarily deployed to storybook-preview February 19, 2024 13:02 Inactive
@github-actions github-actions bot temporarily deployed to storybook-preview February 20, 2024 10:59 Inactive
@lukasoppermann lukasoppermann merged commit 6e59707 into main Feb 20, 2024
26 checks passed
@lukasoppermann lukasoppermann deleted the @lukasoppermann/add-figma-colormodes branch February 20, 2024 11:55
@primer primer bot mentioned this pull request Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants