fix: frontend UI graph rendering on search (#1524)
<!-- .github/pull_request_template.md --> ## Description <!-- Please provide a clear, human-generated description of the changes in this PR. DO NOT use AI-generated descriptions. We want to understand your thought process and reasoning. --> zoomToFit wrapper returned undefined when graph ref wasn't ready ## Type of Change <!-- Please check the relevant option --> - [ ] Bug fix (non-breaking change that fixes an issue) - [ ] New feature (non-breaking change that adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] Documentation update - [ ] Code refactoring - [ ] Performance improvement - [ ] Other (please specify): ## Screenshots/Videos (if applicable) <!-- Add screenshots or videos to help explain your changes --> ## Pre-submission Checklist <!-- Please check all boxes that apply before submitting your PR --> - [ ] **I have tested my changes thoroughly before submitting this PR** - [ ] **This PR contains minimal changes necessary to address the issue/feature** - [ ] My code follows the project's coding standards and style guidelines - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] I have added necessary documentation (if applicable) - [ ] All new and existing tests pass - [ ] I have searched existing PRs to ensure this change hasn't been submitted already - [ ] I have linked any relevant issues in the description - [ ] My commits have clear and descriptive messages ## DCO Affirmation I affirm that all code in every commit of this pull request conforms to the terms of the Topoteretes Developer Certificate of Origin.
This commit is contained in:
commit
a8dab3019e
1 changed files with 16 additions and 2 deletions
|
|
@ -217,10 +217,24 @@ export default function GraphVisualization({ ref, data, graphControls, className
|
||||||
|
|
||||||
const [graphShape, setGraphShape] = useState<string>();
|
const [graphShape, setGraphShape] = useState<string>();
|
||||||
|
|
||||||
|
const zoomToFit: ForceGraphMethods["zoomToFit"] = (
|
||||||
|
durationMs?: number,
|
||||||
|
padding?: number,
|
||||||
|
nodeFilter?: (node: NodeObject) => boolean
|
||||||
|
) => {
|
||||||
|
if (!graphRef.current) {
|
||||||
|
console.warn("GraphVisualization: graphRef not ready yet");
|
||||||
|
return undefined as any;
|
||||||
|
}
|
||||||
|
|
||||||
|
return graphRef.current.zoomToFit?.(durationMs, padding, nodeFilter);
|
||||||
|
};
|
||||||
|
|
||||||
useImperativeHandle(ref, () => ({
|
useImperativeHandle(ref, () => ({
|
||||||
zoomToFit: graphRef.current!.zoomToFit,
|
zoomToFit,
|
||||||
setGraphShape: setGraphShape,
|
setGraphShape,
|
||||||
}));
|
}));
|
||||||
|
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div ref={containerRef} className={classNames("w-full h-full", className)} id="graph-container">
|
<div ref={containerRef} className={classNames("w-full h-full", className)} id="graph-container">
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue