Skip to content
This repository has been archived by the owner on Jul 30, 2023. It is now read-only.

Add styling for mermaid diagrams #1

Open
Cito opened this issue Oct 22, 2020 · 3 comments · Fixed by #6
Open

Add styling for mermaid diagrams #1

Cito opened this issue Oct 22, 2020 · 3 comments · Fixed by #6
Labels
enhancement New feature or request

Comments

@Cito
Copy link

Cito commented Oct 22, 2020

This is a very nice theme. However, it does not play well with mermaid diagrams, because the background and borders of shapes in graphs have the same or similar color as the page background. You can test it with this small diagram:

```mermaid
graph LR
A --> B
```

We could make the background of the diagrams lighter like this:

.mermaid {
    background-color: #555;
}

But it would be better to change only the background and line color of the shapes. Unfortunately, so far I found not simple way to achieve this.

@jamiebrynes7
Copy link
Owner

Ah I never use the mermaid diagrams so I forgot they existed!

I'll have a look at styling these for the next release 😁

@jamiebrynes7 jamiebrynes7 added the enhancement New feature or request label Oct 22, 2020
@jamiebrynes7 jamiebrynes7 changed the title Problem with colors of mermaid graphs Add styling for mermaid diagrams Oct 22, 2020
@Cito
Copy link
Author

Cito commented Oct 25, 2020

Nice - looks much better now. Thank you!

I still found a small issue - the color of arrows in state diagrams.

The color of ER diagrams is also a bit strange. But it's not better in the original theme.

@jamiebrynes7
Copy link
Owner

jamiebrynes7 commented Oct 26, 2020

Oh I may have missed out a few of the arrow headings on the state diagrams. 😕

I haven't done the entity representation or the user journey diagrams yet, I forgot the Git automation would close this issue once the linked PR was merged. 🤦‍♂️


I'll have a look at these things in the next pass 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants