Fix: Inner blocks control position for Cart Block #6973
Fix: Inner blocks control position for Cart Block #6973
Conversation
…inner block controls position.
The release ZIP for this PR is accessible via:
|
Size Change: -37 B (0%) Total Size: 874 kB
ℹ️ 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.
Hi @tarunvijwani! Thank you for fixing the block's control position bug! 🙌 You can find below my comment about a display bug!
assets/js/blocks/cart/edit.js
Outdated
<BlockSettings | ||
attributes={ attributes } | ||
setAttributes={ setAttributes } | ||
/> |
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.
Hi @tarunvijwani! The BlockSettings
component is already added on line 80 of this file. With the current two BlockSettings
we have double dark mode controls:
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.
Sorry, I missed them while resolving the merge conflict. I have removed the duplicate code. Thank you!
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.
That's perfect, @tarunvijwani! It looks good to me! So, let's 🚀 it! 💪
Describe the bug
Cart Block: Order summary inner blocks controls were appearing on the left side of the screen
Changes in the PR
Removed
__experimentalExposeControlsToChildren
fromcart/index.js
andcart/edit.js
file.__experimentalExposeControlsToChildren
was added by the PR to display the view switcher when selecting the inner blocks of the Cart block. It was dependent on PR opened in Gutenberg repo, which was never merged.Currently, the view switcher is only visible when we select the parent Cart block, which is nearly impossible to select without using the list view option. It can be done now using a filter, I'll create a different ticket to track the issue
Fixes #6698
Screenshots
Testing
Automated Tests
User Facing Testing
WooCommerce Visibility
Changelog