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

resolves #310 fix Kroki on Firefox #311

Merged

Conversation

ggrossetie
Copy link
Member

  • Upgrade to asciidoctor-kroki v0.3.0
    • Catch all exceptions when reading a file because Firefox does not allow to read file from file:// protocol for security reasons.
    • Remove rusha from the bundle because for whatever reason, it does not work on Firefox 😞
  • Load kroki.js before the renderer.js because order matters on Firefox.

resolves #310

- Upgrade to asciidoctor-kroki v0.3.0
** Catch all exceptions when reading a file (because Firefox does not allow to read file from file:// protocol for security reasons)
** Remove rusha from the bundle (because for whatever reason, it does not work on Firefox)
- Load kroki.js before the renderer.js (order matters on Firefox)
Because this feature is not supported in a browser environment.
@ggrossetie ggrossetie merged commit af61eb4 into asciidoctor:master Nov 9, 2019
@ggrossetie ggrossetie deleted the issue-310-fix-kroki-firefox branch April 13, 2020 17:15
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

Successfully merging this pull request may close these issues.

On Firefox: diagrams feature enabled - fails
1 participant