-
Notifications
You must be signed in to change notification settings - Fork 28
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
#694: Add Layoutable interface #175
Conversation
Layout Options on children will be useful to configure child-specific layout options (Rather than only be able to configure the parent layout itself). This change also makes the Graph model more consistent with the Sprotty API (Which accepts layout options on SShapeElement) |
Test model: This model can't be open on the current
It can be open with this PR. Note: layout might be off, as I'm still working on the client layout for eclipse-glsp/glsp#694. The goal of this PR is to make it possible to open/parse the model. |
ffe92bb
to
d97e52c
Compare
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.
Thanks Camille, the changes look great and work excellent! 🎉
I have a minor comment inline regarding the year range in the copyrightText of the genModel.
plugins/org.eclipse.glsp.graph/src-gen/org/eclipse/glsp/graph/GLayoutable.java
Outdated
Show resolved
Hide resolved
- Update the Graph model for consistency with the Sprotty API - GShapeElement is a GLayoutable and can have layoutOptions refs eclipse-glsp/glsp#694 Contributed on behalf of STMicroelectronics. Signed-off-by: Camille Letavernier <cletavernier@eclipsesource.com>
d97e52c
to
4f7cb88
Compare
Thanks for the review! I've updated the Genmodel and regenerated the files. I also rebased on master. |
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.
Great, thank you Camille for the update! 👍
cf. #175 Contributed on behalf of STMicroelectronics.
refs eclipse-glsp/glsp#694
Contributed on behalf of STMicroelectronics.