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

Better loading failure mode #148

Open
solarkraft opened this issue Aug 23, 2021 · 1 comment
Open

Better loading failure mode #148

solarkraft opened this issue Aug 23, 2021 · 1 comment

Comments

@solarkraft
Copy link
Contributor

solarkraft commented Aug 23, 2021

Discovered while working on my DokuWiki distribution:

When an error occurs, the promised redirection never happens and, due to the cookie, it will always attempt and fail to open the visual editor.

In the video you see it switching back to the text editor once, I am not sure why.

Screen.Recording.2021-08-23.at.18.40.15-2.mov

Suggested solutions to avoid bricking the editor:

  • Only set the cookie after the editor has successfully loaded
  • Reset the cookie on error
  • Instruct the user to clear cookies (and avoid the visual editor)

The actual error is likely in the way I install the plugin, so I track it in my repo.

@solarkraft
Copy link
Contributor Author

solarkraft commented Oct 12, 2021

The TL;DR solution/workaround for the most common version of this problem is install from the release, not the source (as there is javascript that needs to be compiled).

Maybe the error message should point to an internal error with the possible cause of a wrong installation. It can also be an error in some other plugin's script though, since due to the bundling a single one will take down the JS of the whole page.

The suggestions above still apply, which is why I'm keeping this issue open.

@solarkraft solarkraft changed the title No redirection after error; editor semi-bricked Better loading failure mode Oct 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant