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

Conversation

@Akatori-Design
Copy link
Contributor

@Akatori-Design Akatori-Design commented Jul 13, 2022

Updated all light-theme shadow tokens to match the Storybook Spec.

Description
Added description with "Uses : " examples.

Values:
Updated LG, primary, danger to :

  • LG : 0 2 40 0 #0000001A
  • primary : 0 2 8 0 #037dd633
  • danger : 0 2 8 0 #d73a4966

@Akatori-Design Akatori-Design added color Tokens related to color figma Issues related to figma work labels Jul 13, 2022
@Akatori-Design Akatori-Design requested a review from a team as a code owner July 13, 2022 21:30
@metamaskbot
Copy link
Collaborator

Builds ready [875670f]

@Akatori-Design
Copy link
Contributor Author

reverted MD back to 0 2 16 0

@metamaskbot
Copy link
Collaborator

Builds ready [425e04e]

@Akatori-Design
Copy link
Contributor Author

renamed SM-primary back to primary, SM-danger back to danger

@metamaskbot
Copy link
Collaborator

Builds ready [fd7622b]

@Akatori-Design
Copy link
Contributor Author

fixed danger shadow size to SM 0 2 8 0

@metamaskbot
Copy link
Collaborator

Builds ready [40d23ad]

Copy link
Collaborator

@georgewrmarshall georgewrmarshall left a comment

Choose a reason for hiding this comment

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

One question about error/danger

@metamaskbot
Copy link
Collaborator

Builds ready [782c110]

@metamaskbot
Copy link
Collaborator

Builds ready [54e69f6]

@metamaskbot
Copy link
Collaborator

Builds ready [15cb813]

@georgewrmarshall georgewrmarshall self-requested a review July 21, 2022 22:09
@metamaskbot
Copy link
Collaborator

Builds ready [4778dc6]

"spread": "0"
},
"type": "boxShadow"
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

These are updates from figma so I guess the type has changed for shadow from a number to a string?

@georgewrmarshall georgewrmarshall added the code Issue related to code work label Jul 27, 2022
@georgewrmarshall georgewrmarshall merged commit f4092b2 into main Jul 28, 2022
@georgewrmarshall georgewrmarshall deleted the shadow-updates branch July 28, 2022 23:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

code Issue related to code work color Tokens related to color figma Issues related to figma work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants