mermaid-js / mermaid

Generation of diagrams like flowcharts or sequence diagrams from text in a similar manner as markdown
https://mermaid.js.org
MIT License
71.79k stars 6.51k forks source link

when "self" in a node identifier, using GH mermaid: "Unable to render rich display: Diagram error not found" #5317

Closed ringerc closed 4 months ago

ringerc commented 8 months ago

Description

Mermaid js as adopted by Github's Markdown renderer does not like the string self appearing in a label for a node.

See the two diagrams below.

Steps to reproduce

flowchart TD;

  my_self["me"]
  other["thing"]
  my_self --> other

vs

flowchart TD;

  my_thing["me"]
  other["thing"]
  my_thing --> other

Screenshots

No response

Code Sample

No response

Setup

Suggested Solutions

Additional Context

No response

nirname commented 4 months ago

@ringerc it seems that at the moment both diagrams work just fine at mermaid.live and gh as well. Please, check it and confirm or add some more examples.

ringerc commented 4 months ago

They're rendering here now too. I presume the issue was fixed since when I reported it in Feb. Thanks for checking.