-
Notifications
You must be signed in to change notification settings - Fork 219
Update pattern with images and alignment #9414
Conversation
Noting this should skip testing as this PR is just to fix unreleased changes from #9340 |
The release ZIP for this PR is accessible via:
Script Dependencies ReportThere is no changed script dependency between this branch and trunk. This comment was automatically generated by the TypeScript Errors Report
🎉 🎉 This PR does not introduce new TS errors. |
Size Change: 0 B 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.
Great job! 🚀 Thank you for enhancing this pattern 🙌🏻
By the way, I noticed that the Sprint and Epic fields are empty. Was that done intentionally? 🙂
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.
I was about to merge the changes to include it in the release, but I spotted the indentations are removed from the patterns/featured-category-triple.php
file. Is that the intended change?
@imanish003 - yes that is intentional because if you read my comment above, this is just a fix for an already existing addition to the pattern #9340 which has the sprints and epics. We don't want to have duplicate PRs in this context. @kmanijak - yes that is also intentional because there may be several rounds of changes and it became very tedious to indent them every time we made a change. So I think we could do that later when everything is settled to all patterns at once. |
Fixes #9269
This PR converts the work from above PR that was initially using the product category blocks into using WP core blocks.
Screenshots
Testing
User Facing Testing
WooCommerce Visibility
Performance Impact
Changelog