-
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
|
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.
Unfortunately I don't think you can account for that. We decided to be more opinionated in the design and specific made the text white and button black. However as you can see, themes can still override that design/style. So even if we adjust to fix it for this theme, then you will consequently have issues with other themes. |
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.
Unfortunately I don't think you can account for that. We decided to be more opinionated in the design and specific made the text white and button black. However as you can see, themes can still override that design/style. So even if we adjust to fix it for this theme, then you will consequently have issues with other themes.
Got you. Approving then!
* Update text * Make shop now button black with white text to match design * Update pattern image and text
Closes #9101
This PR updates the image and text used. Secondary update the
Shop now
button coloring to match original design.Note this will not need to be included in the testing instructions.
Screenshots
Testing
User Facing Testing
WooCommerce Visibility
Performance Impact
Changelog