-
-
Notifications
You must be signed in to change notification settings - Fork 8.6k
Open
Labels
Status: TriageNeeds to be verified, categorized, etcNeeds to be verified, categorized, etcType: EnhancementNew feature or requestNew feature or request
Description
Proposal
I use Mermaid extensively to communicate complex topics on my blog.
For consistency, I had to create my own theme. I've been following mermaid theming config only to realize that some stuff can ONLY be done in CSS. Even then, there's no consistency and some elements e.g. some <path> and <span> miss semantic classes so I had to resort to ugly workarounds like using path:nth-child(2) (example) just to get basic theming working.
Unless I'm missing something important, here are a few suggestions:
- Follow the basic mermaid config for ALL diagrams (you can see my config here) so the CSS workarounds aren't needed (e.g. having the actor man the same color as actor
<rect>required a workaround) - Use semantic class names on all elements that the user may want to theme.
- Simplify the generated SVG. There's no reason what could essentially be a
<circle>to be implemented using multiple<path>elements (see the screenshot below from an example state diagram):
Colors
- this is irrelevant as my theme is personal. These are suggestions for the theming systems not to create a new theme
Screenshots
No response
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Status: TriageNeeds to be verified, categorized, etcNeeds to be verified, categorized, etcType: EnhancementNew feature or requestNew feature or request