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

Improved Support for Imported Reactors in Diagrams #1055

Merged
merged 3 commits into from
Mar 30, 2022

Conversation

a-sr
Copy link
Collaborator

@a-sr a-sr commented Mar 20, 2022

A normal selection of diagram elements that originate from code outside the current editor (imported reactors), will now highlight the respective import instead of nothing.
If the Alt key is down during selection of such diagram elements, the active editor will change to the one containing the code. This will also cause the diagram view to update its content this reflect the model of the new active editor.

Closes #889

Since this is a UI specific feature, this current implementation only affects the Epoch IDE. The LF VSC extension requires a separate implementation which I am not familiar with.

a-sr added 3 commits March 20, 2022 14:21
Alt+Click on a diagram element will now jump to the associated code
section even if it is located in a different editor.
This implementation only affects the Epoch IDE.
Selected diagram elements that are associated with code sections outside
the current editor now refer to their import.
@a-sr a-sr added the epoch Epoch IDE label Mar 20, 2022
@edwardalee edwardalee self-assigned this Mar 21, 2022
Copy link
Collaborator

@edwardalee edwardalee left a comment

Choose a reason for hiding this comment

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

Wow, this is more complicated than one would have hoped! But this is great functionality! Let's merge it.

Copy link
Member

@lhstrh lhstrh left a comment

Choose a reason for hiding this comment

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

This definitely requires a level of knowledge about Xtext/Ecore/Eclipse that I don't posses, so my review isn't of much use. The feature is very useful though -- thanks for implementing it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epoch Epoch IDE
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Navigate to imported reactors in diagrams
3 participants