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

subgraph not rendering with change to sample #231

Closed
mycall opened this issue Oct 23, 2015 · 6 comments
Closed

subgraph not rendering with change to sample #231

mycall opened this issue Oct 23, 2015 · 6 comments

Comments

@mycall
Copy link

mycall commented Oct 23, 2015

I am having problems rendering this graph. Am I doing something wrong?

graph TB        
    subgraph old sys 2
        a2(new client)-->b2
        oc2(Old client)-->b2
        a3(test)-->b2
    end

    subgraph old sys 1
        a1(new client)-->b1(sys1 server)
        oc1(Old client)-->b2(test)
    end
@mycall
Copy link
Author

mycall commented Oct 23, 2015

error "Uncaught TypeError: Cannot set property 'label' of undefined" @ mermaid.js:10789

if (!_.has(node, "label") && !g.children(v).length) { node.label = v; }

Are there limitations to subgraphs?

@mycall
Copy link
Author

mycall commented Oct 27, 2015

@mycall
Copy link
Author

mycall commented Oct 30, 2015

any chance you can review the uploaded zip?

@knsv
Copy link
Collaborator

knsv commented Oct 30, 2015

I did and found tab characters to be the problem. Replace with space and
it will work. Mermaid should handle the tabs eventually though.
fre 30 okt. 2015 kl. 18:51 skrev mycall notifications@github.com:

any chance you can review the uploaded zip?


Reply to this email directly or view it on GitHub
#231 (comment).

@mycall
Copy link
Author

mycall commented Oct 30, 2015

Thank you, that solves my problem. I've been using tabs successfully without the subgraph feature but once I added subgraph, I was getting those errors.

Might be an easy regex to fix.

@mycall mycall closed this as completed Oct 30, 2015
mgenereu pushed a commit to mgenereu/mermaid that referenced this issue Jun 25, 2022
…yarn/develop/typescript-eslint/eslint-plugin-4.28.0

Bump @typescript-eslint/eslint-plugin from 4.27.0 to 4.28.0
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

No branches or pull requests

2 participants