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

Site Editor / iframe: Occasional double scrollbar. #30055

Closed
jasmussen opened this issue Mar 20, 2021 · 5 comments
Closed

Site Editor / iframe: Occasional double scrollbar. #30055

jasmussen opened this issue Mar 20, 2021 · 5 comments
Assignees
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Type] Bug An existing feature does not function as intended

Comments

@jasmussen
Copy link
Contributor

Description

When using the site editor, you will occasionally see a double scrollbar. The scrollbar appears to be caused by the sibling inserter being positioned outside of the iframe, but below the content:

bug

Step-by-step reproduction instructions

Preriquisites:

  • Make sure you do not have auto-hiding scrollbars in MacOS
  • Use a block theme, such as TT1 Blocks
  • Open the web-inspector so it's ready to go

Steps to reproduce:

  • Go to the site editor, and while the template is still loading, hover the content to try and invoke the sibling inserter
  • If you're successful, the sibling inserter will be positioned incorrectly, due to being positioned according to content still loading.

Note: If you mange to reproduce, be careful to not invoke the sibling inserter again, or the scrollbar will disappear. One thing that might help: as soon as you reproduce the double scrollbar, move the mouse to the inspector, then open the JS console and paste: setTimeout(function(){debugger;}, 2000);, then wait two seconds. This will lock the content, so you can now scroll down to see the sibling inserter sitting in a gray area outside the content.

Expected behaviour

Only ever a single scrollbar, that of the iframe.

Screenshots or screen recording (optional)

It's super fiddly to get this right, but if you use MacOS auto hiding scrollbars, you'll never see this. So you have to turn that off first.

repro

Code snippet (optional)

One potential fix is to change the CSS as shown in the GIF above, to specifically set overflow: hidden; on .interface-interface-skeleton__content. But because content is placed in the popover container outside the iframe, I'm not convinced that this will actually fix it.

@jasmussen jasmussen added [Type] Bug An existing feature does not function as intended [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") labels Mar 20, 2021
@jasmussen
Copy link
Contributor Author

CC: @ellatrix as I recall you having worked on the iframe.

@annezazu
Copy link
Contributor

Noting that this has come up with WordPress 5.9 as part of the FSE Outreach Program's All Things Media exploration. You can see it replicated at 01:46 in this video: https://app.koofr.net/links/481d3053-4fd1-4756-60ef-5db5b25d0ed2

@jasmussen
Copy link
Contributor Author

@ciampo do you know if #43332 fixed this issue? It seems like it might have! 🙏

@ciampo
Copy link
Contributor

ciampo commented Aug 29, 2022

I believe it should have! It would be great if you could also confirm that you can't repro this issue anymore.

@jasmussen
Copy link
Contributor Author

I haven't seen it in a while! Wonderful, I'll close this one. Thanks again 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Type] Bug An existing feature does not function as intended
4 participants