Remove node backgrounds - Githubissues

multinet-app / multilink

A node-link visualization built to work with the multinet ecosystem.

https://multinet.app

Apache License 2.0

4 stars 2 forks source link

Remove node backgrounds #301

Closed JackWilb closed 2 years ago

JackWilb commented 2 years ago

Does this PR close any open issues?

No

Give a longer description of what this PR addresses and why it's needed

Removes label background from nodes so that the nodes are no longer occluded when a size variable makes them too small to be seen

Provide pictures/videos of the behavior before and after these changes (optional)

Without size applied:

With size applied:

Are there any additional TODOs before this PR is ready to go?

TODOs:

[x] Check in with Alex to see how he feels about this change

netlify[bot] commented 2 years ago

✔️ Deploy Preview for next-multilink ready!

🔨 Explore the source changes: 405b57b0ba4f1a4478ec629a6eba30534147e971

🔍 Inspect the deploy log: https://app.netlify.com/sites/next-multilink/deploys/61fb15508f70c7000898a376

😎 Browse the preview: https://deploy-preview-301--next-multilink.netlify.app/