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

Trailing whitespace at the end of lines is not ignored #55

Closed
vi opened this issue Dec 13, 2014 · 2 comments
Closed

Trailing whitespace at the end of lines is not ignored #55

vi opened this issue Dec 13, 2014 · 2 comments
Labels
Status: Approved Is ready to be worked on

Comments

@vi
Copy link

vi commented Dec 13, 2014

Current master's mermaid.full.js seems to be picky about whitespace at the end of lines. It wasn't so before.

If it's present, the next line shows error like:

Error: Parse error on line 8: ... id2---id1; id2---i -----------------------^ Expecting 'SPACE', 'ALPHA', 'NUM', 'COLON', 'COMMA', 'PLUS', 'EQUALS', 'MULT', 'DOT', 'BRKT', 'STYLE', 'LINKSTYLE', 'CLASSDEF', 'CLASS', 'CLICK', 'PCT', got 'NEWLINE'

throw new Error(str);
mermaid.full.js (line 14585, col 8)

It's 14'th line in my text editor BTW, not 8'th (unless we are counting from the <div> instead from the beginning of html file).

@knsv
Copy link
Collaborator

knsv commented Dec 13, 2014

Thanks for pointing it out. Have pushed a fix along with a test. Will be included in the next release.

@knsv knsv added the Status: Approved Is ready to be worked on label Dec 13, 2014
@knsv
Copy link
Collaborator

knsv commented Dec 15, 2014

Included in release 0.2.16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Approved Is ready to be worked on
Projects
None yet
Development

No branches or pull requests

2 participants