Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Text based labels, new shape #101

Merged
merged 22 commits into from
Jan 14, 2015
Merged

Text based labels, new shape #101

merged 22 commits into from
Jan 14, 2015

Conversation

bjowes
Copy link
Contributor

@bjowes bjowes commented Jan 13, 2015

  • Added the configuration parameter htmlLabels to the mermaid config. It defaults to true, but if set to false, all labels (in nodes and on edges) will be generated as SVG compatible text, not HTML. This resolves issue Generated SVG works poorly outside web browsers #58.
  • Isolated local CSS content to each mermaid SVG. This resolves issue isolate class styling to the svg container #92.
  • Added preliminary reversed asymmetric shape - syntax is not completed though since it conflicts with other elements.
  • Started effort to break out the dagre-d3 dependency, in order to build a truly minimized mermaid. Not completed.

Conflicts:
	dist/mermaid.full.js
	dist/mermaid.full.min.js
	dist/mermaid.slim.js
	dist/mermaid.slim.min.js
	src/diagrams/flowchart/flowRenderer.js
	src/utils.js
	test/web_style.html
Added support to configure mermaid to use plain text labels for better
SVG compatibility
Added a reverse asymmetric shape, not in the graph syntax yet
Better separation of styles between multiple mermaid SVGs on the same
page
Added support for text label styling
Pending resolution of a issue with globals
knsv added a commit that referenced this pull request Jan 14, 2015
Text based labels, new shape
@knsv knsv merged commit 1033a74 into mermaid-js:master Jan 14, 2015
@knsv
Copy link
Collaborator

knsv commented Jan 14, 2015

Thanks!

mgenereu pushed a commit to mgenereu/mermaid that referenced this pull request Jun 25, 2022
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