-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Sankey link/node pixel level alignment #2021
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
Conversation
ccd4785
to
013508b
Compare
@alexcjohnson I tried a number of approaches - including stuff like dropping the path pixel fractions with things like |
@alexcjohnson here are two pens that - for now - point to a custom build with the Does it look fine? |
Also, some screenshot examples: minimal-value version (where you saw the most visible misalignment): https://codepen.io/monfera/full/EwPrKP/ |
Nice, I like it, thanks! 💃 once tests pass. |
f4acf41
to
4d52bbb
Compare
Tweaking the alignment (OOM: one pixel) between nodes and links. For past considerations, see #2017 (comment).
The result of this is generally better alignment, while preserving even box borders and multiedges with little to no overlap artifacts. For examples:
Magnified:

Vertical:

Mike Bostock's example:
