Conversation
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: +28 B (0%) Total Size: 1.1 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.
Looks good!
I checked the banner on various styles and found one caveat. On Canary (TT3 theme) the font is a big bigger and there's not enough width for "40% off". And because of the line-height: 0
it's stack like in the image:
But I understand it's opinionated and should work well in MOST cases, not EVERY case. Hence just mentioning it, but I don't think that's a must.
Also, on designs all the text is left-aligned, while "40% off" seems to have whitespace around it.
Removing them could actually help with the text wrapping in Canary!
Thanks for the review @kmanijak! I've fixed your comments and also added some hardcoded colors that were missing. |
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.
Looks great!
This PRs adds the new
Discount banner
pattern.Fixes #9930
Screenshots
Testing
User-Facing Testing
Discount banner
pattern and save.Shop now
button links to the shop page.WooCommerce Visibility
Changelog