Navigation Menu

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

Navigate to imported reactors in diagrams #889

Closed
lhstrh opened this issue Jan 26, 2022 · 0 comments · Fixed by #1055
Closed

Navigate to imported reactors in diagrams #889

lhstrh opened this issue Jan 26, 2022 · 0 comments · Fixed by #1055

Comments

@lhstrh
Copy link
Member

lhstrh commented Jan 26, 2022

The current behavior is implemented in the klighd framework and it is not really a bug but intended this way.
The basic UX concept is that the editor defines the context for the diagram view. If the selection of a diagram element that is defined in a different resource would jump to the textual source, it would switch to the respective editor and display the diagram for that editor, losing the previous context.
I think this behavior would be annoying because it happens rather often that you select elements without the intent to look at the source code, e.g. to navigate in the model, highlight them for presentation, or expand them.

I could try to add an alternative action, e.g. ctrl+click, to navigate to the source code of reactors in other files. This way, losing context would be intentional but still inconsistent with the default.

@lhstrh I agree, if we want this behavior, we should add a new ticket because this will take some careful consideration in order to provide a pleasant user experience.

Originally posted by @a-sr in #341 (comment)

a-sr added a commit that referenced this issue Mar 20, 2022
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.
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 a pull request may close this issue.

1 participant