-
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
392: Add categories to the model #129
Conversation
f41f7e7
to
89172cf
Compare
480195c
to
784dbb4
Compare
- Add structured compartments using X/Y Layout - Improve VBox Layout to support stacking variable-size compartments - Introduce LayoutData to configure the new VBox layout (and potentially other layouts as well) - Add a custom TypeAdapter for LayoutData to workaround EMF-to-JSon serialization issues - Introduce GeometryUtil to convert from absolute to relative coordinates on the Server Fixes eclipse-glsp/glsp/issues/392 Signed-off-by: Camille Letavernier <cletavernier@eclipsesource.com>
Signed-off-by: Camille Letavernier <cletavernier@eclipsesource.com>
@CamilleLetavernier #127 has been merged so I have rebased your PR to resolve conflicts |
@CamilleLetavernier I also did a quick update of your commit to fix minor code issues. I also removed the new 'layoutData' after talking to Philip. Please see also my comments on the client change. We can discuss this of course but adding yet another layout map/object does not seem necessary to me. |
It looks like there are some inconsistences in the copyright-headers of the generated ecore code. For new files (like |
@tortmayr Great catch! I did as you suggested and re-generated all files and added a new commit. |
Please be aware that there are a few overwritten methods (marked with |
Thanks! LGTM! 👍 |
This PR should be reviewed together with the corresponding client-side changes: eclipse-glsp/glsp-client/pull/136