Skip to content
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

H-3400, H-3401: Graph viz config for node sizing, filtering, colors #5351

Merged
merged 9 commits into from
Oct 8, 2024

Conversation

CiaranMn
Copy link
Member

@CiaranMn CiaranMn commented Oct 8, 2024

🌟 What is the purpose of this PR?

This PR expands the entity graph visualization configuration added in #5242 by:

  1. Allowing users to determine how nodes are sized (their min size, max size, and whether incoming/outgoing/all edges are considered when determining their relative size)
  2. Adds a filter panel, which
    • allows users to choose which types of entities are shown in the graph
    • allows users to set custom colours for nodes of specific types

It also:
5. Increases edge size when a node is hovered/selected
6. Colours edge the same colour as the source node when nodes are highlighted (i.e. they are hovered/selected or are the neighbors of hovered/selected nodes)
7. Enables clicking on a node a second time (moved from first time, which interfered with highlighting behavior) to open the entity in a slideover, in full-screen mode as well as regular mode
8. Avoids unnecessarily laying out the graph when node highlight depth/direction is changed
9. Adds tooltips to all config/filter panels in the graph visualizer

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

  • does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

  • are internal and do not require a docs change

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

  • do not affect the execution graph

vilkinsons
vilkinsons previously approved these changes Oct 8, 2024
@CiaranMn CiaranMn added this pull request to the merge queue Oct 8, 2024
Merged via the queue into main with commit 8f26b28 Oct 8, 2024
35 checks passed
@CiaranMn CiaranMn deleted the cm/more-graph-viz-customization branch October 8, 2024 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/apps > hash* Affects HASH (a `hash-*` app) area/apps type/eng > frontend Owned by the @frontend team
Development

Successfully merging this pull request may close these issues.

2 participants