Open grebaldi opened 1 year ago
We don't want to force users to reload the complete page (F5) when an error occurs. We want devs to be able to use the UI to print debug staff (or to investigate errors without reloading).
Neos.Neos.Ui.ContentReady
stays because we want to introduce a new modal if the iframe indicates a reload@mhsdesign We need to specify what the modal should say and which functionalities to offer.
updateIframeUrlIfNecessary
is fired, we show modalOpened new feature request https://github.com/neos/neos-ui/issues/3746
@grebaldi any thoughts on that?
Things we discussed / found out at the sprint
document.removeEventListener('Neos.Neos.Ui.ContentReady'
is a noop in the iframe and doesnt cleanup the handlerguestNotificationScript
hack, as this is Ui specific)
Description
There are a couple of occasions in which the loading indicator doesn't notice that some async process has failed and just ends up looping endlessly.
Several older issues mention this phenomenon:
Steps to Reproduce
Repro 1
./flow session:destroyall
Repro 2
Neos.Fusion:Debug
output like this:Repro 3
Expected behavior
The loading indicator should not run in an endless loop, but stop when an error occurs.
Actual behavior
Affected Versions
Neos: >= 7.3 (and older)
UI: >= 7.3 (and older)