-
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
Recent regression - Site Editor - Spinner component not visible / missing CSS. #28153
Comments
I wonder if it may be related to the new Iframe? #25775 cc @ellatrix @youknowriad if you may have other ideas about what may have caused this. |
Sorry for the late reply. What stylesheet should be loaded that isn't loaded right now? |
No worries, thanks for taking a look!
After poking around briefly, it looks like it is the styles defined in JS that correspond to what were missing in the screenshots above: gutenberg/packages/components/src/spinner/styles/spinner-styles.js Lines 27 to 34 in c6b43a0
|
@ItsJonQ Did you know more about these generated styles and how to insert them into a different document? |
Haiii!!! Sorry for the late reply @ellatrix . For these styles, what needs to happen is they need to be rendered in a Previously, I created a library to help with this exact problem: We can use the same technique for the iFrame editor as well |
Describe the bug
The loading spinner does not show up as expected in the site editor. It seems to be missing CSS.
Currently - the spinner component is not visible and seems to be missing styles:
As compared to previously when it did show up, these styles were present:
I noticed this as a recent regression while testing related to the bug where template parts would not resolve when themes are installed in subdirectories - #28088. Drafted Sunday evening, this PR was showing the spinner. After rebasing
master
this afternoon, the spinners seem to have disappeared as their CSS is no longer present - so it seems a recent change has caused this.The specific spinner I have noted comes from
gutenberg/packages/block-library/src/template-part/edit/index.js
Line 127 in 9c9617d
The text was updated successfully, but these errors were encountered: