-
-
Notifications
You must be signed in to change notification settings - Fork 187
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
Direct association #2526
Direct association #2526
Conversation
for more information, see https://pre-commit.ci
@sz332 This looks good. Is it ready to merge? |
It is ready to merge, the one think I don't know how to do is translation. Should I do something, or is it a separate process? If I need to do something, can you please help me to add the necessary translation string? |
Translation is something we kick of separately mostly, so we can update all text in one shot. |
@sz332 The unit tests fail with a message:
I think the keyboard shortcut should be removed from directed-association. It can be |
Removed the keyboard shortcuts. Can you try it out? Thanks! |
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
Direct association between classes and sysml blocks.
What is the current behavior?
Currently it is very painful to create a direct (navigable) association. A normal association must be created, then going into properties, finding the tail element, and setting navigability. It takes just too much time.
Issue Number: 2516
What is the new behavior?
A new toolbar button is added enabling the same outcome with a single click.
Does this PR introduce a breaking change?
Other information
I could not find how to properly do the translations so I need some help in that.