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

UI lib - new theme colors #7375

Merged
merged 6 commits into from
Aug 14, 2022
Merged

UI lib - new theme colors #7375

merged 6 commits into from
Aug 14, 2022

Conversation

pettinarip
Copy link
Member

@pettinarip pettinarip commented Aug 10, 2022

Defines base theme colors.

Description

  1. Implements new Design System palette
  2. It also defines the old colors used in the old theme so that we can migrate the components to chakra and keep the same styles as before. In next iterations, we are going to remove them and only use colors from the Design System palette.

Design System colors: https://www.figma.com/file/NrNxGjBL0Yl1PrNrOT8G2B/ethereum.org-Design-System

Related Issue

Fixes #7374
Epic #6374

@gatsby-cloud
Copy link

gatsby-cloud bot commented Aug 10, 2022

Gatsby Cloud Build Report

ethereum-org-website-dev

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 13m

@pettinarip pettinarip changed the title Theme colors UI lib - new theme colors Aug 10, 2022
// define old colors from the old theme as semanticTokens to transition
// from emotion components to chakra
// TODO: remove these colors as we migrate away from them
...oldColors,
Copy link
Member Author

Choose a reason for hiding this comment

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

This is to do the migration as seamless as possible. Detailed in the migration guide under the ## Theme colors header. #7368
The idea is that we keep using the same color variable names.

Copy link
Member

@corwintines corwintines left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

@pettinarip pettinarip merged commit d3f7940 into dev Aug 14, 2022
@pettinarip pettinarip deleted the theme-colors branch August 14, 2022 17:33
@corwintines corwintines mentioned this pull request Aug 15, 2022
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.

Define base theme colors
2 participants