-
Notifications
You must be signed in to change notification settings - Fork 525
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
render: outside connection label masking #447
render: outside connection label masking #447
Conversation
I'm not sure.. |
If we need to make the association of outside connection labels clearer that sounds like a separate but related issue to work on. This is solving the legibility of the text itself. I don't think we want to use the fact that the connection can be seen through the text as a way to reason that the label does not belong to that particular connection. Updated the screenshot to show the full example, not just the zoomed in portion and it is clearer which connection the label belongs to. |
I get it and I agree that it is more about positioning then the mask. Can you provide some larger examples, with more edges and other overlaps so that we know how it would look like? |
with #1383 I think this can have the same behavior as outside shape labels: |
c020640
to
3ad501e
Compare
3ad501e
to
ef60dec
Compare
Summary
We only added label masks when the label is placed over the connection, but when TALA places labels above or below connections the labels could still overlap other connections at times.
This fixes the issue by adding transparent label masks for outside labels (to match shape labels) so that even if the label is over part of another connection the mask will still be applied for improved legibility.
Details
fabricated example where the
111111
label is in an outside position with a mask over another connectionbefore/after