Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Widgets screen block toolbar overlaps the Block Area accordion #16765

Conversation

jorgefilipecosta
Copy link
Member

Fix: #16602

This PR adds small padding to the block list container for widget areas to make sure the block toolbars don't overlap the block area accordion.

How has this been tested?

I went to the block widgets screen.
I added a block like an image block and verified the block toolbar does not overlap the block area accordion.

@jorgefilipecosta jorgefilipecosta added [Type] Bug An existing feature does not function as intended [Feature] Widgets Screen The block-based screen that replaced widgets.php. labels Jul 26, 2019
Copy link
Contributor

@draganescu draganescu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There still seems to be some overlap for me in Safari:

Screenshot 2019-08-05 at 15 20 01

@jorgefilipecosta jorgefilipecosta force-pushed the fix/widgets-screen-block-toolbar-overlaps-the-block-area-accordion-trigger branch from 4a5e61f to 93390bf Compare August 5, 2019 15:45
@jorgefilipecosta
Copy link
Member Author

Hi @draganescu, thank you for the review the dimensions were updated it should work as expected now.

@mapk
Copy link
Contributor

mapk commented Aug 5, 2019

I just tested and no longer see an overlap in FF or in Chrome. Thanks! This looks good to 🚢 .

Screen Shot 2019-08-05 at 2 59 59 PM

@jorgefilipecosta jorgefilipecosta merged commit 71db908 into master Aug 5, 2019
@jorgefilipecosta jorgefilipecosta deleted the fix/widgets-screen-block-toolbar-overlaps-the-block-area-accordion-trigger branch August 5, 2019 23:04
@youknowriad youknowriad added this to the Gutenberg 6.3 milestone Aug 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Widgets Screen The block-based screen that replaced widgets.php. [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Widgets: The block toolbar should not overlap the Block Area accordion trigger
4 participants