Skip to content

Docs: MDC callouts everywhere, and fail the build on a Nunjucks tag - #5890

Merged
Yndira-E merged 1 commit into
mainfrom
docs/mdc-callouts
Sep 29, 2026
Merged

Yndira-E merged 1 commit into
mainfrom
docs/mdc-callouts

Conversation

@dimitrieh

Copy link
Copy Markdown
Contributor

Description

Follow-up to FlowFuse/flowfuse#8694, which moved the product docs to MDC callouts. The website's own pages under /docs still used the old ff-callout HTML, so they now use ::note, ::warning and ::caution too, and the conversion code and styles are removed.

A leftover {% %} tag in the flowfuse docs now fails the build with the file and line, rather than being stripped silently. The silent strip is what hid the installation form on /docs/install/introduction/.

Related Issue(s)

FlowFuse/flowfuse#8694

Checklist

  • I have read the contribution guidelines
  • I have considered the performance impact of these changes
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
  • For blog PRs, an Art Request has been created (instructions)

FlowFuse/flowfuse#8694 moved the product docs from Eleventy shortcodes to
::note and ::warning, which render as Nuxt UI callouts. The website's own
pages under /docs still hardcoded the ff-callout HTML the docs import used
to generate, so the two styles sat side by side.

- Convert the hardcoded callouts in content-guides to ::note, ::warning
  and ::caution. The OPC UA caution had no blank lines, so its body was
  raw HTML and its bold text rendered as literal asterisks.
- NodeRedHelp used ff-callout--info, which never had a CSS rule, so the
  note rendered unstyled. It is a ProseNote now.
- Drop convertCallouts, stripTemplateEngineOverride and the ff-callout
  styles, which nothing uses any more.
- Replace the silent strip of leftover {% %} tags with a build error that
  names the file and line. The strip is what hid the missing installation
  form on /docs/install/introduction/. Fenced and inline code are exempt.
@netlify

netlify Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for flowfuse-website ready!

Name Link
🔨 Latest commit 8a719b4
🔍 Latest deploy log https://app.netlify.com/projects/flowfuse-website/deploys/6abbb808bdf5210008f3019c
😎 Deploy Preview https://deploy-preview-5890--flowfuse-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 39 (🟢 up 8 from production)
Accessibility: 95 (no change from production)
Best Practices: 92 (no change from production)
SEO: 92 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@Yndira-E
Yndira-E merged commit 6014aaf into main Sep 29, 2026
7 checks passed
@Yndira-E
Yndira-E deleted the docs/mdc-callouts branch September 29, 2026 16:42
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.

2 participants