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.

Sign up for GitHub

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

Jump to bottom

Navigation / Page List: Remove Placeholder around spinner #38981

Closed
jasmussen opened this issue Feb 22, 2022 · 0 comments · Fixed by #38983
Closed

Navigation / Page List: Remove Placeholder around spinner #38981

jasmussen opened this issue Feb 22, 2022 · 0 comments · Fixed by #38983
Assignees
@Mamaduka
Labels
[Block] Navigation Affects the Navigation Block [Block] Page List Affects the Page List Block [Type] Enhancement A suggestion for improvement.

Comments

@jasmussen
Copy link
Contributor

When the Page List block loads, it shows a spinner inside a bordered box:
page list

The bordered box appears to be a Placeholder container. If we remove this, the spinner should be able to sit on its own. That would mitigate some of the visual glitching mentioned in #38970.
@jasmussen jasmussen added [Type] Enhancement A suggestion for improvement. [Block] Navigation Affects the Navigation Block [Block] Page List Affects the Page List Block labels Feb 22, 2022
@jasmussen jasmussen mentioned this issue Feb 22, 2022
Merged
8 tasks
@Mamaduka Mamaduka mentioned this issue Feb 22, 2022
Merged
8 tasks
@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label Feb 22, 2022
@priethor priethor removed the [Status] In Progress Tracking issues with work in progress label May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Mamaduka Mamaduka

Labels
[Block] Navigation Affects the Navigation Block [Block] Page List Affects the Page List Block [Type] Enhancement A suggestion for improvement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Page List: Simplify loading state WordPress/gutenberg
3 participants
@Mamaduka @jasmussen @priethor