-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Comments
CC: @ellatrix as I recall you having worked on the iframe. |
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 |
I believe it should have! It would be great if you could also confirm that you can't repro this issue anymore. |
I haven't seen it in a while! Wonderful, I'll close this one. Thanks again 🙏 |
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:
Step-by-step reproduction instructions
Preriquisites:
Steps to reproduce:
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.
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.The text was updated successfully, but these errors were encountered: