Skip to content

Adding all color modes to figma export - #805

Merged
lukasoppermann merged 9 commits into
mainfrom
@lukasoppermann/add-figma-colormodes
Feb 20, 2024
Merged

lukasoppermann merged 9 commits into
mainfrom
@lukasoppermann/add-figma-colormodes

Conversation

@lukasoppermann

Copy link
Copy Markdown
Collaborator

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, pksjce and rezrah February 16, 2024 14:26
@changeset-bot

changeset-bot Bot commented Feb 16, 2024 •

Copy link
Copy Markdown

🦋 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

@github-actions

github-actions Bot commented Feb 16, 2024 •

Copy link
Copy Markdown
Contributor
Variables changed
No variables changed

Comment thread scripts/buildPlatforms/buildFigma.ts Outdated
//
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
Copy Markdown
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
Copy Markdown
Collaborator 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
Copy Markdown
Collaborator 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.

@github-actions

Copy link
Copy Markdown
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
@lukasoppermann
lukasoppermann deleted the @lukasoppermann/add-figma-colormodes branch February 20, 2024 11:55
@primer primer Bot mentioned this pull request Feb 20, 2024

This branch was previously deployed

2 inactive deployments
storybook-preview — 882384aa Deployed Feb 20, 2024 by github-actions[bot]
github-pages — 882384aa Deployed Feb 20, 2024 by lukasoppermann
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