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

Align tokens with Figma variables #8310

Closed
2 of 6 tasks
alisonailea opened this issue Dec 1, 2023 · 3 comments
Closed
2 of 6 tasks

Align tokens with Figma variables #8310

alisonailea opened this issue Dec 1, 2023 · 3 comments
Assignees
Labels
4 - verified Issues that have been released and confirmed resolved. breaking change Issues and pull requests with code changes that are not backwards compatible. Calcite (dev) Issues logged by Calcite developers. calcite-design-tokens Issues specific to the @esri/calcite-design-tokens package. enhancement Issues tied to a new feature or request. estimate - 2 Small fix or update, may require updates to tests. impact - p1 - need for current milestone User set priority impact status of p1 - need for current milestone p - high Issue should be addressed in the current milestone, impacts component or core functionality

Comments

@alisonailea
Copy link
Contributor

Check existing issues

Description

Found some misalignments between Figma variables and tokens.

Acceptance Criteria

Calcite Token names and values should align with the Calcite Figma variables

Relevant Info

No response

Which Component

Core and Semantic tokens

Example Use Case

No response

Priority impact

p1 - need for current milestone

Calcite package

  • @esri/calcite-components
  • @esri/calcite-components-angular
  • @esri/calcite-components-react
  • @esri/calcite-design-tokens
  • @esri/eslint-plugin-calcite-components

Esri team

Calcite (dev)

@alisonailea alisonailea added enhancement Issues tied to a new feature or request. 0 - new New issues that need assignment. needs triage Planning workflow - pending design/dev review. labels Dec 1, 2023
@alisonailea alisonailea self-assigned this Dec 1, 2023
@alisonailea alisonailea added 2 - in development Issues that are actively being worked on. and removed 0 - new New issues that need assignment. labels Dec 1, 2023
@github-actions github-actions bot added impact - p1 - need for current milestone User set priority impact status of p1 - need for current milestone Calcite (dev) Issues logged by Calcite developers. labels Dec 1, 2023
@alisonailea alisonailea added this to the 2023 November Priorities milestone Dec 1, 2023
@github-actions github-actions bot added the calcite-design-tokens Issues specific to the @esri/calcite-design-tokens package. label Dec 1, 2023
@alisonailea alisonailea added p - high Issue should be addressed in the current milestone, impacts component or core functionality estimate - 2 Small fix or update, may require updates to tests. labels Dec 1, 2023
@geospatialem geospatialem added breaking change Issues and pull requests with code changes that are not backwards compatible. and removed needs triage Planning workflow - pending design/dev review. labels Dec 1, 2023
alisonailea added a commit that referenced this issue Dec 1, 2023
**Related Issue:** #8310

## Summary

There were some misalignments between figma variables and tokens as well
as some changes to names to better align with the naming schema.

Core tokens are a logical scale
Semantic tokens have meaningful names representing their usage.

BREAKING CHANGE:

- Box-shadow is now shadow
- Core font family tokens use the font name
- Remove unnecessary core tokens line-height, font-size, letter-spacing,
paragraph-spacing as these can be exclusive to semantic and reference
core size tokens
- Core size tokens now use their pixel size in their name
- Border-radius is now corner-radius
- Border-width removed unnecessary tokens none, sm, md, lg
- Success, info, warning, and danger colors are now prefixed with the
name "status"
  -   --color-status-info-hover
benelan pushed a commit that referenced this issue Dec 1, 2023
**Related Issue:** #8310

## Summary

There were some misalignments between figma variables and tokens as well
as some changes to names to better align with the naming schema.

Core tokens are a logical scale
Semantic tokens have meaningful names representing their usage.

BREAKING CHANGE:

- Box-shadow is now shadow
- Core font family tokens use the font name
- Remove unnecessary core tokens line-height, font-size, letter-spacing,
paragraph-spacing as these can be exclusive to semantic and reference
core size tokens
- Core size tokens now use their pixel size in their name
- Border-radius is now corner-radius
- Border-width removed unnecessary tokens none, sm, md, lg
- Success, info, warning, and danger colors are now prefixed with the
name "status"
  -   --color-status-info-hover
benelan pushed a commit that referenced this issue Dec 1, 2023
**Related Issue:** #8310

## Summary

There were some misalignments between figma variables and tokens as well
as some changes to names to better align with the naming schema.

Core tokens are a logical scale
Semantic tokens have meaningful names representing their usage.

BREAKING CHANGE:

- Box-shadow is now shadow
- Core font family tokens use the font name
- Remove unnecessary core tokens line-height, font-size, letter-spacing,
paragraph-spacing as these can be exclusive to semantic and reference
core size tokens
- Core size tokens now use their pixel size in their name
- Border-radius is now corner-radius
- Border-width removed unnecessary tokens none, sm, md, lg
- Success, info, warning, and danger colors are now prefixed with the
name "status"
  -   --color-status-info-hover
benelan pushed a commit that referenced this issue Dec 2, 2023
**Related Issue:** #8310

## Summary

There were some misalignments between figma variables and tokens as well
as some changes to names to better align with the naming schema.

Core tokens are a logical scale
Semantic tokens have meaningful names representing their usage.

BREAKING CHANGE:

- Box-shadow is now shadow
- Core font family tokens use the font name
- Remove unnecessary core tokens line-height, font-size, letter-spacing,
paragraph-spacing as these can be exclusive to semantic and reference
core size tokens
- Core size tokens now use their pixel size in their name
- Border-radius is now corner-radius
- Border-width removed unnecessary tokens none, sm, md, lg
- Success, info, warning, and danger colors are now prefixed with the
name "status"
  -   --color-status-info-hover
@jcfranco
Copy link
Member

jcfranco commented Dec 6, 2023

@alisonailea I went ahead and marked this one as installed for v2.

@jcfranco jcfranco added 3 - installed Issues that have been merged to master branch and are ready for final confirmation. and removed 2 - in development Issues that are actively being worked on. labels Dec 6, 2023
@github-actions github-actions bot assigned geospatialem and DitwanP and unassigned alisonailea Dec 6, 2023
Copy link
Contributor

github-actions bot commented Dec 6, 2023

Installed and assigned for verification.

@geospatialem geospatialem added 4 - verified Issues that have been released and confirmed resolved. and removed 3 - installed Issues that have been merged to master branch and are ready for final confirmation. labels Dec 6, 2023
@geospatialem
Copy link
Member

Verified in calcite-design-tokens in 2.0.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4 - verified Issues that have been released and confirmed resolved. breaking change Issues and pull requests with code changes that are not backwards compatible. Calcite (dev) Issues logged by Calcite developers. calcite-design-tokens Issues specific to the @esri/calcite-design-tokens package. enhancement Issues tied to a new feature or request. estimate - 2 Small fix or update, may require updates to tests. impact - p1 - need for current milestone User set priority impact status of p1 - need for current milestone p - high Issue should be addressed in the current milestone, impacts component or core functionality
Projects
None yet
Development

No branches or pull requests

4 participants