Skip to content

Conversation

@dervoeti
Copy link
Member

@dervoeti dervoeti commented Nov 5, 2025

MermaidJS was not loaded, because stackabletech/documentation-ui#139 removed header-scripts and the Mermaid extension is loaded in the header-scripts template by default. We could change script_stem to head-scripts (what we use) but the Mermaid extension seems to remove all scripts in that section instead of just appending the Mermaid script.
So I added loading of a separate mermaid-scripts partial in head.hbs (in stackabletech/documentation-ui#140) and configured the script_stem in the extension configuration.

@netlify
Copy link

netlify bot commented Nov 5, 2025

Deploy Preview for stackable-docs ready!

Name Link
🔨 Latest commit dfb4c9c
🔍 Latest deploy log https://app.netlify.com/projects/stackable-docs/deploys/690b8405d51267000805e12b
😎 Deploy Preview https://deploy-preview-796--stackable-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@dervoeti dervoeti added this pull request to the merge queue Nov 6, 2025
Merged via the queue into main with commit a460244 Nov 6, 2025
7 checks passed
@dervoeti dervoeti deleted the fix/mermaid-js-loading branch November 6, 2025 08:00
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.

3 participants