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

underscores in labels are no longer supported #915

Closed
alistair-broomhead opened this issue Aug 28, 2019 · 6 comments
Closed

underscores in labels are no longer supported #915

alistair-broomhead opened this issue Aug 28, 2019 · 6 comments
Projects

Comments

@alistair-broomhead
Copy link

alistair-broomhead commented Aug 28, 2019

Describe the bug
Upon upgrading from 8.2.3 to 8.2.4 you can no longer use underscores in labels. The results in links to the live editor created last week now leading to an error page

To Reproduce
Steps to reproduce the behavior:
Create a graph using the following:

graph LR

foo[my_lib.foo]
bar[my_lib.bar]
baz[my_lib.qux.baz]

foo --> bar
bar --> baz

Expected behavior
Screenshot 2019-08-28 at 11 06 56

Screenshots
Screenshot 2019-08-28 at 11 08 50

Desktop (please complete the following information):

  • OS: OSX
  • Browser Chrome
  • Version 75

Additional context
The reproduction screenshots were taken using local checkouts of the live editor repo, but
the issue is live at mermaidjs.github.io/mermaid-live-editor

@JonMidhir
Copy link

Confirming that we've experienced this as well after upgrading from 8.2.3 to 8.2.4.

A workaround is to replace underscores (_) with colons (:) however this potentially requires a project-wide update.

@knsv
Copy link
Collaborator

knsv commented Aug 28, 2019

Will fix it in 8.2.6 but meanwhile you can simply put the string within quotes like below:

graph LR

foo["my_lib.foo"]
bar["my_lib.bar"]
baz["my_lib.qux.baz"]

foo --> bar
bar --> baz

@knsv knsv added this to In progress in 8.2.0 Aug 28, 2019
@JonMidhir
Copy link

@knsv Actually I was mistaken, our error is slightly different from the OP. We get an error when using underscores in class names in 8.2.4. In 8.2.5 it looks like this goes away but occurs instead when using the class name as the first argument to click.

image

Should I create a separate issue for this or do you think these are related and will both be fixed in 8.2.6?

@stanhu
Copy link
Contributor

stanhu commented Aug 28, 2019

@knsv Thanks! We just ran into this too and will look for 8.2.6.

@alistair-broomhead
Copy link
Author

@knsv Thanks, and kudos on such a quick response/turn-around

@knsv
Copy link
Collaborator

knsv commented Sep 1, 2019

Released in 8.2.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
8.2.0
  
Released
Development

No branches or pull requests

4 participants