Skip to content

Conversation

@drewnoakes
Copy link
Contributor

The size of the target graph is often so large that related nodes aren't visible within the viewport.

This change adds a checkbox to the toolbar that causes nodes that aren't directly referenced from the selected node. Clicking on one of those nodes will refocus the graph around that node, adding any newly needed nodes and hiding any no longer directly referenced.

Screencast (includes command from #894, though that's not in this PR):

direct-references-only

The size of the target graph is often so large that related nodes aren't visible within the viewport.

This change adds a checkbox to the toolbar that causes nodes that aren't directly referenced from the selected node. Clicking on one of those nodes will refocus the graph around that node, adding any newly needed nodes and hiding any no longer directly referenced.
Copy link
Owner

@KirillOsenkov KirillOsenkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is amazing!!

Now do the cones (closure of all referenced and closure of all referencing) :)

@KirillOsenkov KirillOsenkov merged commit 9f65014 into KirillOsenkov:main Sep 14, 2025
1 check passed
@drewnoakes drewnoakes deleted the direct-references-only branch September 14, 2025 11:20
@drewnoakes
Copy link
Contributor Author

Now do the cones (closure of all referenced and closure of all referencing) :)

Great idea. Any idea or preference on how to present that in the UI?

@KirillOsenkov
Copy link
Owner

maybe another checkbox ("show transitive closure only") or something

@drewnoakes
Copy link
Contributor Author

Ok, so one checkbox to support both directions (one option for both cones rather than two options for separate cones). Will give it a crack.

@drewnoakes
Copy link
Contributor Author

drewnoakes commented Sep 15, 2025

#898

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants