Skip to content
This repository has been archived by the owner on Nov 26, 2024. It is now read-only.

Tokens v2 - DO NOT MERGE #669

Closed
wants to merge 11 commits into from
Closed

Tokens v2 - DO NOT MERGE #669

wants to merge 11 commits into from

Conversation

garrettbear
Copy link
Contributor

Description

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

  • I’ve followed MetaMask Coding Standards.
  • I've clearly explained what problem this PR is solving and how it is solved.
  • I've linked related issues
  • I've included manual testing steps
  • I've included screenshots/recordings if applicable
  • I’ve included tests if applicable
  • I’ve documented my code using JSDoc format if applicable
  • I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.
  • I’ve properly set the pull request status:
    • In case it's not yet "ready for review", I've set it to "draft".
    • In case it's "ready for review", I've changed it from "draft" to "non-draft".

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@metamaskbot
Copy link
Collaborator

Builds ready [ad7e561]

Storybook: Storybook

@metamaskbot
Copy link
Collaborator

Builds ready [fede13f]

Storybook: Storybook

@metamaskbot
Copy link
Collaborator

Builds ready [5970eb6]

Storybook: Storybook

@metamaskbot
Copy link
Collaborator

Builds ready [ec8e864]

Storybook: Storybook

@metamaskbot
Copy link
Collaborator

Builds ready [c0e46df]

Storybook: Storybook

@metamaskbot
Copy link
Collaborator

Builds ready [4ebeed1]

Storybook: Storybook

@metamaskbot
Copy link
Collaborator

Builds ready [44c6c7f]

Storybook: Storybook

georgewrmarshall and others added 6 commits May 21, 2024 12:17
* chore: inital add of blank json file

```
- Blue updates
 - blue000 no longer exist
 - blue025 added
 - blue 050 added

- Orange updates
 - orange000 no longer exist
 - orange 025 added
 - orange 050 added

- Green updates
 - green000 no longer exist
 - green025 added
 - green 050 added

- Red updates
 - red000 no longer exist
 - red025 added
 - red050 added

- Yellow updates
 - yellow000 no longer exist
 - yellow025 added
 - yellow050 added
 - yellow700 added
 - yellow800 added
 - yellow900 added

- Violet no longer exist

- Purple added

- Lime added
```

---------

Co-authored-by: georgewrmarshall <[email protected]>
* feat: updated cssvariables

* refactor: added comments to colors

* refactor: flatlisted design tokens

* refactor: removed spacing in design tokens

* Update design-tokens.css

* Update src/css/shadow.css

Co-authored-by: George Marshall <[email protected]>

---------

Co-authored-by: George Marshall <[email protected]>
@metamaskbot
Copy link
Collaborator

Builds ready [ad9b943]

Storybook: Storybook

* feat: use figma light theme variables

* feat: update tests

* feat: working version of using brandEvolution

* feat: working version of using brandEvolution

* feat: dark mode update

* feat: update useJsonColors functions

* remove console.log

* feat: update tests

* fix lint issue

* lint update

* all tests passing

* remove unused imports

* Feat/theme tokens brand evolution (#686)

* feat: add brand evolution themes

* brand evo themes

* change export

* support functionality for white and black brand colors

* update dark theme background test

* lint fix

* use js for stories

* clean up unushed props

* revert to old js mapping for stories
automation-scripts/themeColors.js Fixed Show resolved Hide resolved
@metamaskbot
Copy link
Collaborator

Builds ready [915917d]

Storybook: Storybook

* feat: updated css color variables

* feat: updated migration doc with token changes

* fix: updated comments in css variables

* refactor: updated spacing

* feat: added descriptions as comments to themed tokens

* fix: removed newline

* fix: updated css format

* fix: updated lime800 color

* fix: removed root from dark colors

* adding line breaks in migration doc

---------

Co-authored-by: georgewrmarshall <[email protected]>
@metamaskbot
Copy link
Collaborator

Builds ready [762facd]

Storybook: Storybook

@metamaskbot
Copy link
Collaborator

Builds ready [06d80da]

Storybook: Storybook

@metamaskbot
Copy link
Collaborator

Builds ready [e71ad5d]

Storybook: Storybook

@metamaskbot
Copy link
Collaborator

Builds ready [3f72760]

Storybook: Storybook

@georgewrmarshall
Copy link
Collaborator

Closing in favor of #698 and #692

@georgewrmarshall georgewrmarshall deleted the tokens-v2 branch May 23, 2024 20:03
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