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

Fix Custom Palette colors and support multiple origins and theme cache issues #4588

Merged
merged 3 commits into from
Feb 16, 2022

Conversation

geriux
Copy link
Contributor

@geriux geriux commented Feb 16, 2022

Fixes WordPress/gutenberg#38525 and WordPress/gutenberg#36152

This is a follow-up of WordPress/gutenberg#38474

Description

This PR fixes several issues, first, it fixes an ongoing issue where changing between themes would not refresh the current theme data and a restart of the editor was needed, as well as sometimes opening the editor for the first time and not seeing the theme colors.

Second, it adds support for multiple color and gradient palettes including the Custom ones.

To test check the Gutenberg PR description.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Feb 16, 2022

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

@fluiddot
Copy link
Contributor

I saw that unit tests are also failing in #4586, so looks like we have an actual breakage here due to recent changes in Gutenberg trunk branch.

@fluiddot
Copy link
Contributor

saw that unit tests are also failing in #4586, so looks like we have an actual breakage here due to recent changes in Gutenberg trunk branch.

Heads up that I'm working on a fix (reference).

@geriux
Copy link
Contributor Author

geriux commented Feb 16, 2022

Heads up that I'm working on a fix (reference).

Thanks for working on the fix for that 🎉 I updated this PR and it will only update the release notes since the Gutenberg reference was already updated 👍

@fluiddot
Copy link
Contributor

Thanks for working on the fix for that 🎉 I updated this PR and it will only update the release notes since the Gutenberg reference was already updated 👍

I was about to notify you that the fix is already merged 😅, glad to know that it worked.

Copy link
Contributor

@fluiddot fluiddot left a comment

Choose a reason for hiding this comment

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

@fluiddot fluiddot added this to the 1.72.0 milestone Feb 16, 2022
@geriux geriux merged commit e08069c into develop Feb 16, 2022
@geriux geriux deleted the feature/support-multiple-color-palettes branch February 16, 2022 14:39
mokagio added a commit to wordpress-mobile/WordPress-iOS that referenced this pull request Mar 7, 2022
The PR originally linked was closed,
wordpress-mobile/gutenberg-mobile#4537 ,
in favor of the new one,
wordpress-mobile/gutenberg-mobile#4588 .

While it was easy enough to find the latter by reading through the
conversation in the former, I decided to bypass that to remove friction.
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.

Support multiple color palettes on Mobile
2 participants