-
-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
Broken CLI Graphs? (v0.5.1) #196
Comments
I think this is a result of lacking styling. I could replicate your issue and eliminating it by adding |
Is this the way it's supposed to work? Is there any way the css could just be included by default if no css file is given? I feel that this issue would happen to just about everyone who's starting out by reading the documentation online. |
I definitely got caught up by this, and I imagine many others will. Is there a reason why we shouldn't just use the default styles when generating a PNG file? Or is this a recent regression? It seems like the default behaviour, which is to generate blank PNG files, is confusing. |
I agree! @fardog is this something you could find time for? I am unsure how the paths to the css files would be calculated in the CLI when installed globally. |
I've sent a PR to address the issue, tested it locally by using |
I'll review the posted PR after work and give it a +1 if it looks good |
Something's happened since I last used the CLI. Has the website not been updated, or is everything breaking? Even something as simple as this:
or
Is giving me a completely blank PNG...not even a background. Just completely transparent with nothing shown.
Also, is this (from #153 ):
supposed to give this .png?
Where is that extra
A
element coming from? I tried putting a semi-colon after thestyle
line, but nothing changed.Mermaid & PhantomJS are both updated to the newest version.
The text was updated successfully, but these errors were encountered: