marcel/mermaid
1
0
Fork 0
mirror of https://github.com/mermaid-js/mermaid.git synced 2025-10-08 08:39:38 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: fix personBorder/Bkg C4 diagram theme vars

Browse Source 
The current `personBorder`/`personBkg` theme variables for C4 diagrams
are set to the string `'calculated'`.

However, despite being `'calculated'`, they never seem to change to
anything else, and so become invalid CSS variables.

I've instead changed these to just default to base theme vars,
as that's what they do in [`these-base.js`][1].

[1]: 727bf30824/packages/mermaid/src/themes/theme-base.js (L106-L107)
This commit is contained in:
Alois Klink
2023-04-13 05:48:02 +01:00
parent 83d7d6c48f
commit 616d370a51
4 changed files with 8 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
packages/mermaid/src/themes/theme-dark.js
View File

@@ -81,9 +81,8 @@ class Theme {
this.todayLineColor = '#DB5757';
/* C4 Context Diagram variables */
this.personBorder = 'calculated';
this.personBkg = 'calculated';
this.personBorder = this.primaryBorderColor;
this.personBkg = this.mainBkg;
/* state colors */
this.labelColor = 'calculated';

5
packages/mermaid/src/themes/theme-default.js
View File

@@ -109,9 +109,8 @@ class Theme {
this.todayLineColor = 'red';
/* C4 Context Diagram variables */
this.personBorder = 'calculated';
this.personBkg = 'calculated';
this.personBorder = this.primaryBorderColor;
this.personBkg = this.mainBkg;
/* state colors */
this.labelColor = 'black';

5
packages/mermaid/src/themes/theme-forest.js
View File

@@ -83,9 +83,8 @@ class Theme {
this.todayLineColor = 'red';
/* C4 Context Diagram variables */
this.personBorder = 'calculated';
this.personBkg = 'calculated';
this.personBorder = this.primaryBorderColor;
this.personBkg = this.mainBkg;
/* state colors */
this.labelColor = 'black';

5
packages/mermaid/src/themes/theme-neutral.js
View File

@@ -95,9 +95,8 @@ class Theme {
this.todayLineColor = 'calculated';
/* C4 Context Diagram variables */
this.personBorder = 'calculated';
this.personBkg = 'calculated';
this.personBorder = this.primaryBorderColor;
this.personBkg = this.mainBkg;
/* state colors */
this.labelColor = 'black';