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

Bugfix/3477 infinite loading indicator #3714

Draft
wants to merge 5 commits into
base: 8.3
Choose a base branch
from

Conversation

mhsdesign
Copy link
Member

@mhsdesign mhsdesign commented Feb 7, 2024

Resolves: #3477

Alternate fix replaces #2566 which is good and we need to make the page handler non neos ui specific so it can go "back" to neos

The initial idea to spin until the ui informs to be ready was that if possibly a lot of ckeditors have to be initialised and that the content manager should wait until the indicator stops spinning before clicking around.
IN my problems i could not verify that and as this is just an indicator it does no harm stopping a few milliseconds earlier. It never prevented anything or is used for anything critical.
In fact the infinite spinning indicator is much worse when developing and debugging

What I did

How I did it

How to verify it

@mhsdesign mhsdesign changed the base branch from 9.0 to 8.3 February 7, 2024 07:42
@github-actions github-actions bot added 9.0 8.3 and removed 9.0 labels Feb 7, 2024
@mhsdesign mhsdesign marked this pull request as draft February 9, 2024 16:15
@neos-bot
Copy link

🎥 End-to-End Test Recordings

These videos demonstrate the end-to-end tests for the changes in this pull request.

@mhsdesign
Copy link
Member Author

@73nici

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: Occasions of the loading indicator running in an endless loop
2 participants