From b7e9d02b7cc4b297d21a5d0810af1f39781b8e51 Mon Sep 17 00:00:00 2001 From: darshanr0107 Date: Tue, 26 Aug 2025 18:10:39 +0530 Subject: [PATCH] fix: ensure configs from initialize and frontmatter are both handled correctly on-behalf-of: @Mermaid-Chart --- packages/mermaid/src/config.spec.ts | 12 ++++++------ packages/mermaid/src/config.ts | 8 ++++---- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/packages/mermaid/src/config.spec.ts b/packages/mermaid/src/config.spec.ts index 7fbae03af..7d43ef0f5 100644 --- a/packages/mermaid/src/config.spec.ts +++ b/packages/mermaid/src/config.spec.ts @@ -82,6 +82,7 @@ describe('when working with site config', () => { describe('getUserDefinedConfig', () => { beforeEach(() => { configApi.reset(); + configApi.saveConfigFromInitialize({}); }); it('should return empty object when no user config is defined', () => { @@ -106,7 +107,6 @@ describe('getUserDefinedConfig', () => { expect(configApi.getUserDefinedConfig()).toMatchInlineSnapshot(` { - "fontFamily": "Arial", "fontSize": 14, "layout": "elk", "theme": "forest", @@ -128,8 +128,8 @@ describe('getUserDefinedConfig', () => { { "fontFamily": "Arial", "fontSize": 14, - "layout": "elk", - "theme": "forest", + "layout": "dagre", + "theme": "dark", } `); }); @@ -152,7 +152,7 @@ describe('getUserDefinedConfig', () => { { "flowchart": { "curve": "basis", - "nodeSpacing": 75, + "nodeSpacing": 50, "rankSpacing": 100, }, "mindmap": { @@ -196,8 +196,8 @@ describe('getUserDefinedConfig', () => { expect(userConfig).toMatchInlineSnapshot(` { "flowchart": { - "curve": "basis", - "nodeSpacing": 100, + "curve": "linear", + "nodeSpacing": 50, "rankSpacing": 100, }, "fontSize": 12, diff --git a/packages/mermaid/src/config.ts b/packages/mermaid/src/config.ts index 4fcb3224d..ce46b993b 100644 --- a/packages/mermaid/src/config.ts +++ b/packages/mermaid/src/config.ts @@ -252,13 +252,13 @@ const checkConfig = (config: MermaidConfig) => { export const getUserDefinedConfig = (): MermaidConfig => { let userConfig: MermaidConfig = {}; - if (configFromInitialize) { - userConfig = assignWithDepth(userConfig, configFromInitialize); - } - for (const d of directives) { userConfig = assignWithDepth(userConfig, d); } + if (configFromInitialize) { + userConfig = assignWithDepth(userConfig, configFromInitialize); + } + return userConfig; };