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

Flowchart library #423

Merged
merged 3 commits into from Feb 21, 2024
Merged

Flowchart library #423

merged 3 commits into from Feb 21, 2024

Conversation

gfontorbe
Copy link
Contributor

Adds a new package sprotty-library which serves as a collection of views with the aim of making easier to start a diagramming solution based on widely used diagrams.

The first part of the new package implements views and interfaces for dealing with flowchart diagrams.

It also provides an inversify ContainerModule where node and edges types are link to the provided views.

Screenshot 2024-02-02 at 11 47 36

Signed-off-by: Guillaume Fontorbe <guillaume.fontorbe@typefox.io>
Copy link
Contributor

@spoenemann spoenemann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

Remark about the previously merged #416: I think examples/flowchart/src/model-source.ts is not necessary. You can simply pass the model in to the updateModel call in standalone.ts.

packages/sprotty-library/package.json Outdated Show resolved Hide resolved
packages/sprotty-library/src/index.ts Outdated Show resolved Hide resolved
Signed-off-by: Guillaume Fontorbe <guillaume.fontorbe@typefox.io>
Copy link
Contributor

@spoenemann spoenemann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, just one improvement:

packages/sprotty-library/src/flowchart/container.ts Outdated Show resolved Hide resolved
Signed-off-by: Guillaume Fontorbe <guillaume.fontorbe@typefox.io>
@gfontorbe
Copy link
Contributor Author

Added a more specific model element for each element in the container.

We can replace that when we have a free form anchor computation in place

Copy link
Contributor

@spoenemann spoenemann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@gfontorbe gfontorbe merged commit 06967c1 into master Feb 21, 2024
2 checks passed
@gfontorbe gfontorbe deleted the gfontorbe/flowchart-library branch February 21, 2024 14:24
@spoenemann spoenemann added this to the v1.2.0 milestone Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants