-
Notifications
You must be signed in to change notification settings - Fork 219
Conversation
The release ZIP for this PR is accessible via:
Script Dependencies ReportThe
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
|
I'm not sure what to do with that, it's not great, it's unlikely anyone will choose the pattern without the preview. The only way I see right now it's not to use the
Maybe for this we can try to use the |
If we don't use the single product block then I am not seeing this pattern as very useful other than giving them an visual idea which then we don't actually need a pattern and a simple example documentation would do. If we used only WP core blocks, they would have to remove almost everything to either put in a single product block or add their own individual parts of single product block (which I don't think we support?). WDYT? cc @shaunandrews Also note that I believe the Single Product Block is still in experimental and many things are not ironed out yet. Perhaps when it is complete, it will have more options to manipulate the image size/ratio... show/hide quantity etc? |
This PR has been marked as If deemed still relevant, the pr can be kept active by ensuring it's up to date with the main branch and removing the stale label. |
I've updated the pattern to use WP core blocks instead of the Single Product Block. |
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.
Good job! 👏
Fixes #9109
This PR adds the
Product Hero
pattern.Note that the image supplied is more portrait than landscape so it will not look exactly like the one #9109
Screenshots
Testing
User Facing Testing
WooCommerce Visibility
Performance Impact
Changelog