-
Notifications
You must be signed in to change notification settings - Fork 2k
Try: general color scales in theme package #102612
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
Try: general color scales in theme package #102612
Conversation
Jetpack Cloud live (direct link)
Automattic for Agencies live (direct link)
|
|
This PR does not affect the size of JS and CSS bundles shipped to the user's browser. Generated by performance advisor bot at iscalypsofastyet.com. |
|
This PR modifies the release build for the following Calypso Apps: For info about this notification, see here: PCYsg-OT6-p2
To test WordPress.com changes, run |
|
This PR has been marked as stale. This happened because:
If this PR is still useful, please do a trunk merge or rebase and otherwise make sure it's up to date and has clear testing instructions. You may also want to ping possible reviewers in case they've forgotten about it. Please close this PR if you think it's not valid anymore — if you do, please add a brief explanation. If the PR is not updated (or at least commented on) in another month, it will be automatically closed. |
Requires #102611
Closes ARC-90
Proposed Changes
This PR tests color generation in the theme package:
primarycolor, afuncoefficient (ie. how saturated should grays be), and ascheme(ie. light or dark mode)The Storybook example can be spinned up locally by running the following command:
yarn workspace @automattic/theme run storybook:startWhy are these changes being made?
Testing Instructions
Pre-merge Checklist