Skip to content

fix(mermaid): resolve CSS theme colors before initialization - #279

Merged
wang-kaopu merged 2 commits into
omdsh-dev:mainfrom
stvsl:fix/mermaid-resolved-theme-colors
Oct 11, 2026
Merged

wang-kaopu merged 2 commits into
omdsh-dev:mainfrom
stvsl:fix/mermaid-resolved-theme-colors

Conversation

@stvsl

@stvsl stvsl commented Oct 10, 2026

Copy link
Copy Markdown
Contributor

在 Whale Song 等皮肤下,语法正确的 Mermaid 图表会因为主题颜色包含 calc() 而渲染失败,并被误报为图表语法错误。例如:

Unsupported color format: "rgba(235, 242, 250, calc(.4 + 0 * .4))"

getComputedStyle(document.body).getPropertyValue(...) 返回的 CSS 自定义属性仍可能包含计算表达式。这个改动先通过临时 DOM 元素解析真实颜色,再将需要转换的现代 CSS 颜色转为 sRGB,使 Mermaid 初始化时接收到受支持的 RGB/RGBA。透明度和原有备用颜色保持有效,初始化期间缓存重复使用的主题颜色。中英文降级提示改为“渲染失败”。

新增颜色回归测试,并同步现有渲染状态测试的提示文案。版本号、依赖声明、发布流程和 Mermaid 安全配置保持原样。

验证:

  • pnpm run check:TypeScript 检查、完整 Vitest 测试套件和 tsdown 构建通过;新增的 13 个颜色用例全部通过。
  • Chromium:原始 0.11.4 构建可复现上述 calc() 错误;修复分支的构建通过默认/Whale Song 明暗主题、不同透明度、现代颜色和无效 token 共 8 组用例,原始 17 节点图正常渲染。
  • 浏览器检查同时验证临时节点清理、SVG 安全校验和无效 Mermaid 语法仍被拒绝。

@wang-kaopu
wang-kaopu merged commit cf8131d into omdsh-dev:main Oct 11, 2026
13 of 14 checks passed
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