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

support mermaid stateDiagram syntax #126

Open
forivall opened this issue Aug 6, 2020 · 3 comments
Open

support mermaid stateDiagram syntax #126

forivall opened this issue Aug 6, 2020 · 3 comments

Comments

@forivall
Copy link

forivall commented Aug 6, 2020

from a quick glance, it's unclear if state-machine-cat is a superset of mermaid-js. It would be nice, if so, if it could parse mermaidjs state diagrams as a language.

Context

it would just be neat, especially since i'm used to mermaid's syntax

Proposed Behavior

it would be a new input type

Current Behavior

supported input are smcat, json and scxml

Considered alternatives

well, just using mermaid, of course.

@forivall forivall changed the title Comparison with mermaid stateDiagram support mermaid stateDiagram syntax Aug 6, 2020
@sverweij
Copy link
Owner

sverweij commented Aug 6, 2020

Hi @forivall state-machine-cat is not a super set of mermaid, syntax wise.

I hadn't looked into mermaid for a few years and wasn't even aware they had state machine support right now, so thanks for the heads-up.

Feature wise state-machine-cat currently likely is a superset of mermaid; from a quick glance mermaid doesn't seem to support history states, terminations, junctions, activities, conditions, actions and events yet. It does seem to have some stuff that influence layout (right_of, left_of), though, and that's something smc does not have - and it might have a more flexible/ better render engine (not tried yet).

That said transforming mermaid syntax into state-machine-cat's internal format (json schema here) should be doable. PR welcome.

@forivall
Copy link
Author

forivall commented Aug 6, 2020

Yeah I definitely was thinking superset, feature-wise. Deffo not syntax. 👍

@stale
Copy link

stale bot commented Aug 16, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale label Aug 16, 2020
@sverweij sverweij removed the stale label Aug 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants