-
Notifications
You must be signed in to change notification settings - Fork 219
Add border support to mini cart contents #8654
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: +95 B (0%) Total Size: 1.12 MB
ℹ️ View Unchanged
|
f205a47
to
13a9553
Compare
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.
LGTM! I'm not sure if we should use experimental API without the feature plugin flag.
@@ -41,6 +41,10 @@ const settings: BlockConfiguration = { | |||
link: true, | |||
}, | |||
lock: false, | |||
__experimentalBorder: { |
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.
Is it correct that we enable this feature in WC Core too? I'm asking because it is an experimental API
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.
Yes, you are right, it shouldn't. I'll fix it. Thanks :)
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.
Pushed the fix, can you have another look? @gigitux
318eecb
to
e46216b
Compare
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.
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.
LGTM
This PR adds support for adding border style controls to the Mini Cart overlay via the Mini Cart Contents block.
Fixes #6257
Screenshots
Testing
Automated Tests
User Facing Testing
Mini Cart
block to the header.Mini Cart
template.Mini Cart contents
block and check that in the settings sidebar you can see theBorder
controls.+
button and add a border and save the template.Mini Cart
button, and check you see the same border in the editor and in the frontend.WooCommerce Visibility
Changelog