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

Adds Custom Color Controls to Customizer for Quadrat #3817

Closed
wants to merge 4 commits into from

Conversation

pbking
Copy link
Contributor

@pbking pbking commented May 10, 2021

Accomplishes the color customization of #3661

This uses the 'overwrite CSS variables' method to customize values.

These customizations are ONLY available in the Customizer.

The things to be customized are defined in the theme.json's 'custom_meta' block.

The 'Customizer Bridge' presents controls to the user based on that JSON and renders the stored values as Global Styles CSS variables.

The user DOES NOT have access to the selected colors via the block editor panel.

image

image

@pbking pbking linked an issue May 10, 2021 that may be closed by this pull request
@pbking pbking added this to the Quadrat v1 milestone May 10, 2021
@MaggieCabrera
Copy link
Contributor

This works fine in the customizer and the frontend but not on the editor, and we are also missing the new colors from the block options (they are showing the old palette)

@pbking
Copy link
Contributor Author

pbking commented May 11, 2021

Still not convinced this is the right approach but now I believe that all of the bugs and notes noted have been resolved except for what is noted in the description.

@jffng
Copy link
Contributor

jffng commented May 20, 2021

Nice exploration @pbking — could we close or remove this from the milestone in favor of #3856?

@pbking
Copy link
Contributor Author

pbking commented May 20, 2021

Yes, absolutely. It was a worthwhile experiment. :)

@pbking pbking closed this May 20, 2021
@pbking pbking deleted the try/quadrat-customize-3 branch September 13, 2021 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants