-
Notifications
You must be signed in to change notification settings - Fork 48
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
feat: add support for a open link #51
feat: add support for a open link #51
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@igorwessel thank you for the contribution.
Could you make the 👇🏻 changes ?
- Could you give descriptive names for the test case similar to how its in https://github.com/excalidraw/mermaid-to-excalidraw/blob/master/playground/testcases/sequence.ts ?
- Also update the Changelog.md
78e0787
to
9be28da
Compare
@ad1992 I've applied the same format to all the flowchart tests. Could you take a look when you get a chance? I wasn't sure about the last ones, so I labeled them as 'Complex' for now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Pushed some minor tweaks.
Thanks @igorwessel ✨
Closes excalidraw/excalidraw#7872
Closes #40
Mermaid provides a synyax to represent a link without arrow head.
This Pull Request intends to introduce a feature that removes arrow heads when a specific syntax is encountered.
Test Cases