Conversation
Size Change: +1.52 kB (0%) Total Size: 1.11 MB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice solution here Albert 👍 (and great sleuthing).
I confirmed the select is fixed as described. I also reviewed our other blocks and ensured interfaces weren't affected by this change.
Let's 🚢 🎉
src/Assets.php
Outdated
// In the editor, we want our stylesheets to load after editor CSS (see #3068). | ||
// This might be no longer necessary when https://github.com/WordPress/gutenberg/issues/20797 | ||
// is merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we create an issue for the backlog for removing this extra code when the fix in Gutenberg lands and we can safely remove because the minimum version of WordPress we support has the fix (if so, you could convert this comment to a todo)?
Remove fix to load our stylesheets after editor CSS. See ...Remove fix to load our stylesheets after editor CSS. See #3068 for the rationale of this fix. It should be no longer github.com/WordPress/gutenberg/issues/20797). https://github.com/woocommerce/woocommerce-gutenberg-products-block/blob/e07de12ed23db91cd977be4423132f3151573d87/src/Assets.php#L39-L51🚀 This comment was generated by the automations bot based on a
|
Could this also be causing the Full Site Editing page in Gutenberg to be completely blank as reported in WordPress/gutenberg#27938? I'm using WooCommerce 4.8.0. I don't have the WooCommerce Blocks plugin installed. |
Hi @chvillanuevap, thanks for raising this! I created a new issue so we can investigate it: #3626. |
Fixes #3068.
The issue was caused because editor CSS was leaking into the block. It could be fixed ensuring our CSS is loaded after
wp-edit-post-css
. Notice this fix might no longer be needed if WordPress/gutenberg#20797 gets merged into Gutenberg.Screenshots
Before:
After:
How to test the changes in this Pull Request:
Notice the semi-transparent background is not fixed since it's a different issue (#2487).
Changelog