-
Notifications
You must be signed in to change notification settings - Fork 11
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
[ELK] Improve routing style management when launching an "arrange all" using an ELK layout #333
Labels
bugzilla
Issues migrated from the old Eclipse Bugzilla
component: diagrams
type: enhancement
Making an existing feature better
Milestone
Comments
lredor
added a commit
that referenced
this issue
Mar 26, 2024
This commit just migrates the data used by SimpleELKLayoutTest without any change. It also seems that some diagrams have not been refreshed according to previous modifications done on VSM file. All diagrams have been opened (and so refreshed) for this commit. Bug: #333
lredor
added a commit
that referenced
this issue
Mar 26, 2024
- 5 new representations have been added in tests data: - one with oblique edges, - one with rectilinear edges, - one with a mix of them and same number of oblique and rectilinear edges, - one with a mix of them and more oblique edges, - and one with a mix of them more rectilinear edges. - 5 tests corresponding to these representations have been added to check the routing style before and after the arrange all with ELK. Bug: #333
lredor
added a commit
that referenced
this issue
Mar 26, 2024
During the conversion from DDiagram to ELK Graph, the routing styles of original Sirius DEdge is now stored on the common parent of edges in ELK Graph. This property can be overridden by defining the routing style directly in the VSM. The tests have also been adapted. Indeed, before this commit the routing applied was always Rectilinear. Now, it is not the case. The existing tests have been "duplicated", to have the previous behavior (by forcing the "Rectilinear routing" in the VSM) and the new behavior for all existing tests by considering the initial used routing style. Bug: #333
lredor
added a commit
that referenced
this issue
Mar 26, 2024
Before this commit a StringValueStyle was created due to a bug in ELK that creates an empty list of junction points. This also causes, as side effect, an edge that seems to be "customized". Bug: #333
lredor
added a commit
that referenced
this issue
Mar 26, 2024
The representations created before the previous commit, that fixes the unexpected creation of StringValueStyle, can contain some kind of "empty StringValueStyle". The goal of this commit is to remove them. This commit also adds tests to check migration effect. Bug: #333
lredor
added a commit
that referenced
this issue
Jun 20, 2024
This commit just migrates the data used by SimpleELKLayoutTest without any change. It also seems that some diagrams have not been refreshed according to previous modifications done on VSM file. All diagrams have been opened (and so refreshed) for this commit. Bug: #333
lredor
added a commit
that referenced
this issue
Jun 20, 2024
- 5 new representations have been added in tests data: - one with oblique edges, - one with rectilinear edges, - one with a mix of them and same number of oblique and rectilinear edges, - one with a mix of them and more oblique edges, - and one with a mix of them more rectilinear edges. - 5 tests corresponding to these representations have been added to check the routing style before and after the arrange all with ELK. Bug: #333
lredor
added a commit
that referenced
this issue
Jun 20, 2024
During the conversion from DDiagram to ELK Graph, the routing styles of original Sirius DEdge is now stored on the common parent of edges in ELK Graph. This property can be overridden by defining the routing style directly in the VSM. The tests have also been adapted. Indeed, before this commit the routing applied was always Rectilinear. Now, it is not the case. The existing tests have been "duplicated", to have the previous behavior (by forcing the "Rectilinear routing" in the VSM) and the new behavior for all existing tests by considering the initial used routing style. Bug: #333
lredor
added a commit
that referenced
this issue
Jun 20, 2024
Before this commit a StringValueStyle was created due to a bug in ELK that creates an empty list of junction points. This also causes, as side effect, an edge that seems to be "customized". Bug: #333
lredor
added a commit
that referenced
this issue
Jun 20, 2024
The representations created before the previous commit, that fixes the unexpected creation of StringValueStyle, can contain some kind of "empty StringValueStyle". The goal of this commit is to remove them. This commit also adds tests to check migration effect. Bug: #333
lredor
added a commit
that referenced
this issue
Jun 26, 2024
This commit just migrates the data used by SimpleELKLayoutTest without any change. It also seems that some diagrams have not been refreshed according to previous modifications done on VSM file. All diagrams have been opened (and so refreshed) for this commit. Bug: #333
lredor
added a commit
that referenced
this issue
Jun 26, 2024
- 5 new representations have been added in tests data: - one with oblique edges, - one with rectilinear edges, - one with a mix of them and same number of oblique and rectilinear edges, - one with a mix of them and more oblique edges, - and one with a mix of them more rectilinear edges. - 5 tests corresponding to these representations have been added to check the routing style before and after the arrange all with ELK. Bug: #333
lredor
added a commit
that referenced
this issue
Jun 26, 2024
During the conversion from DDiagram to ELK Graph, the routing styles of original Sirius DEdge is now stored on the common parent of edges in ELK Graph. This property can be overridden by defining the routing style directly in the VSM. The tests have also been adapted. Indeed, before this commit the routing applied was always Rectilinear. Now, it is not the case. The existing tests have been "duplicated", to have the previous behavior (by forcing the "Rectilinear routing" in the VSM) and the new behavior for all existing tests by considering the initial used routing style. Bug: #333
lredor
added a commit
that referenced
this issue
Jun 26, 2024
Before this commit a StringValueStyle was created due to a bug in ELK that creates an empty list of junction points. This also causes, as side effect, an edge that seems to be "customized". Bug: #333
lredor
added a commit
that referenced
this issue
Jun 26, 2024
The representations created before the previous commit, that fixes the unexpected creation of StringValueStyle, can contain some kind of "empty StringValueStyle". The goal of this commit is to remove them. This commit also adds tests to check migration effect. Bug: #333
lredor
added a commit
that referenced
this issue
Jun 26, 2024
This commit just migrates the data used by SimpleELKLayoutTest without any change. It also seems that some diagrams have not been refreshed according to previous modifications done on VSM file. All diagrams have been opened (and so refreshed) for this commit. Bug: #333
lredor
added a commit
that referenced
this issue
Jun 26, 2024
- 5 new representations have been added in tests data: - one with oblique edges, - one with rectilinear edges, - one with a mix of them and same number of oblique and rectilinear edges, - one with a mix of them and more oblique edges, - and one with a mix of them more rectilinear edges. - 5 tests corresponding to these representations have been added to check the routing style before and after the arrange all with ELK. Bug: #333
lredor
added a commit
that referenced
this issue
Jun 26, 2024
During the conversion from DDiagram to ELK Graph, the routing styles of original Sirius DEdge is now stored on the common parent of edges in ELK Graph. This property can be overridden by defining the routing style directly in the VSM. The tests have also been adapted. Indeed, before this commit the routing applied was always Rectilinear. Now, it is not the case. The existing tests have been "duplicated", to have the previous behavior (by forcing the "Rectilinear routing" in the VSM) and the new behavior for all existing tests by considering the initial used routing style. Bug: #333
lredor
added a commit
that referenced
this issue
Jun 26, 2024
Before this commit a StringValueStyle was created due to a bug in ELK that creates an empty list of junction points. This also causes, as side effect, an edge that seems to be "customized". Bug: #333
lredor
added a commit
that referenced
this issue
Jun 26, 2024
The representations created before the previous commit, that fixes the unexpected creation of StringValueStyle, can contain some kind of "empty StringValueStyle". The goal of this commit is to remove them. This commit also adds tests to check migration effect. Bug: #333
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bugzilla
Issues migrated from the old Eclipse Bugzilla
component: diagrams
type: enhancement
Making an existing feature better
As explain in the bugzilla 538933, the edge routing is currently not kept when launching a layout with ELK.
Steps to reproduce (with a more complete use case than bugzilla 538933):
First technical analysis:
In the scope of bugzilla 548820 - "[ELK] Improve ELK graph transformation", a comment has been added in GmfLayoutEditPolicy to say that the routing style is on the parent edges's and not on each edges. In the scope of this bugzilla 548820, the routing style is set after the layout according to the routing style property value of the root element. But it is not automatically set according the routing styles of the edges in the diagram.
The goal of this issue is to kept, as much as possible, the initial edge routing style defined in Sirius.
The text was updated successfully, but these errors were encountered: