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

Document Actions: Block Toolbar overlaps Document Titles #25803

Closed
jeyip opened this issue Oct 3, 2020 · 5 comments
Closed

Document Actions: Block Toolbar overlaps Document Titles #25803

jeyip opened this issue Oct 3, 2020 · 5 comments
Labels
[Type] Bug An existing feature does not function as intended

Comments

@jeyip
Copy link
Contributor

jeyip commented Oct 3, 2020

Describe the bug
When scrolling through the block editor, if a block toolbar is displayed, it can overlap the document actions labels.

To reproduce
Steps to reproduce the behavior:

  1. Set up the site editor locally
  2. Navigate to the site editor
  3. Hover over a sibling inserter
  4. Scroll down through the editor until the block toolbar would overlap the document actions labels
  5. Verify that there is no visual overlap

Expected behavior
When scrolling through the block editor, the block toolbar does not overlap document actions labels.

Screenshots
2020-10-02 17 27 24

@jeyip jeyip added [Type] Bug An existing feature does not function as intended [Feature] Full Site Editing labels Oct 3, 2020
@Addison-Stavlo
Copy link
Contributor

I noticed the block-popover has z-index of 31, while the header in the interface skeleton has z-index of 30. It seems to be noted as intended to be above the header:

// Above the block list and the header.
".block-editor-block-list__block-popover": 31,

However, it does not behave this way in the post editor. I tried to dig around a bit but am unsure why it behaves differently in the other editor. 🤔

@jeyip
Copy link
Contributor Author

jeyip commented Oct 14, 2020

@shaunandrews

I see that you introduced the code Addie mentioned above. Link to the commit and the pr. Do you happen to remember why we wanted the popover above the block list and the header?

I'm making my way through the PR (it's pretty long) and, I might be missing something, but I can't seem to find more information about the reasoning 😅

@shaunandrews
Copy link
Contributor

Mainly so the parent selector button would be visible.

@noahtallen noahtallen removed their assignment Dec 9, 2020
@david-szabo97
Copy link
Member

This seems to be resolved by now:

Screen.Recording.2021-02-25.at.14.52.40.mov

@shaunandrews
Copy link
Contributor

Can confirm; This has been fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

5 participants