-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Fix outlines #2134
Fix outlines #2134
Conversation
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.
Testing this change I can see that the connection outline no longer shows for connections:
This may be a fallout of our "improving selection outline for connections" end of last year, cf. modeler showing only the bendpoints for connections (which does not work in the viewer case):
To be considered:
- Why do we show selection (outline) in the first place? I'd expect the viewer to not ship with visual selection. I'm fine with "hidden selection", i.e. a selectable state that can be reacted to by integrations.
I tested this change without the diagram styles, the recommended way to embed the viewer. It works with and without the outline module, however I found #2135. Again this sparks the question how much we gain from having selection UI in the viewer. |
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.
Works fine in recommended viewer distribution => #2134 (comment).
Let's target this at |
Custom outlines were only included in modeler.
Closes #2133