Skip to content

Unify graph controls with panel styling - #217

Merged
vogelsgesang merged 3 commits into
mainfrom
custom-graph-toolbar
Oct 9, 2026
Merged

vogelsgesang merged 3 commits into
mainfrom
custom-graph-toolbar

Conversation

@vogelsgesang

Copy link
Copy Markdown
Collaborator

Graph controls now match the rest of the application's panels and use
the same border, border-radius, backdrop, and the same custom tooltips
as other icon buttons.
Now that we use our own styles, we can switch React Flow to its
structural base stylesheet instead of its full stylesheet.

Remove the unnecessary zoom buttons. They provide little value, everyone
just zooms in/out using the mouse.

Share the panel surface colors, borders, corner radii, and shadows
between the control group, settings, collapsible panels, and active
graph nodes.

Graph controls now match the rest of the application's panels and use
the same border, border-radius, backdrop, and the same custom tooltips
as other icon buttons.
Now that we use our own styles, we can switch React Flow to its
structural base stylesheet instead of its full stylesheet.

Remove the unnecessary zoom buttons. They provide little value, everyone
just zooms in/out using the mouse.

Share the panel surface colors, borders, corner radii, and shadows
between the control group, settings, collapsible panels, and active
graph nodes.
Comment thread query-graphs/src/ui/QueryGraph.tsx Outdated
Comment thread query-graphs/src/ui/QueryGraph.css
Comment thread query-graphs/src/ui/QueryGraph.css Outdated
Comment thread query-graphs/src/ui/QueryGraph.css Outdated
Pass the graph toolbar through QueryGraphInternal's children so the internal renderer no longer needs to know about additional application controls.

Size toolbar icons through their buttons and use one filled-icon treatment across the toolbar, removing icon-specific styling.
Comment thread query-graphs/src/ui/QueryGraph.tsx Outdated
Remove the additionalControls slot from QueryGraph. Callers can now place the exported GraphToolbar directly among the graph's children and compose application-specific buttons inside it.
@vogelsgesang
vogelsgesang merged commit 5fae6ea into main Oct 9, 2026
5 checks passed
@vogelsgesang
vogelsgesang deleted the custom-graph-toolbar branch October 9, 2026 12:57
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.

1 participant