Skip to content
This repository was archived by the owner on Apr 13, 2022. It is now read-only.

Update color variants to match Figma #108

Merged
merged 4 commits into from
Feb 26, 2020
Merged

Conversation

sarahill
Copy link
Contributor

We're working to stabilize colors between tokens and Figma. This PR adjust the lightness values of some variants to match the visual style.

This spreadsheet will die once the changes are made. Using this to compare values and document updates as I go.
https://docs.google.com/spreadsheets/d/1VtebqhKrVD9l90O4veYFcH86D9xiZxeuSt-IWTew6yc/edit?usp=sharing

@kaelig kaelig temporarily deployed to polaris-toke-update-col-bmiezd February 25, 2020 16:19 Inactive
@kaelig kaelig temporarily deployed to polaris-toke-update-col-bmiezd February 25, 2020 17:17 Inactive
@sarahill sarahill requested a review from tmlayton February 25, 2020 17:17
@sarahill sarahill changed the title [WIP] Update color variants to match Figma Update color variants to match Figma Feb 25, 2020
@sarahill sarahill requested a review from martenbjork February 25, 2020 17:35
@martenbjork
Copy link
Contributor

@danrosenthal What's a good workflow for quickly getting seeing the final tokens, calculated using the adjustments made in this PR? I.e. In the same format as this file.

I've tried both this branch and next locally but none of the build scripts seem to generate the right tokens file.

@kaelig kaelig temporarily deployed to polaris-toke-update-col-bmiezd February 25, 2020 20:12 Inactive
@sarahill sarahill requested review from kyledurand and tmlayton and removed request for tmlayton, danrosenthal and kyledurand February 25, 2020 20:12
@kaelig kaelig temporarily deployed to polaris-toke-update-col-bmiezd February 25, 2020 22:18 Inactive
@sarahill sarahill merged commit 3255f7b into master Feb 26, 2020
@sarahill sarahill deleted the update-color-variants branch February 26, 2020 11:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants