-
Notifications
You must be signed in to change notification settings - Fork 8
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
Graph Navigator #603
base: master
Are you sure you want to change the base?
Graph Navigator #603
Conversation
|
Failed to generate code suggestions for PR |
179580a
to
8ce784d
Compare
const innerGraph = node['_innerGraph']; | ||
if (!innerGraph) return null; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know how we can get this type out of packages/graph-engine/src/nodes/generic/subgraph.ts
so we can do a proper typescript type check.
I would have liked to do a check like
if (node instanceof SubgraphNode) {
const innerGraph = node._innerGraph;
if (!innerGraph) return null;
}
but I'm not sure how importing for types/classes works for such packages
if (typeof window !== 'undefined') { | ||
window['newGraphEditor'] = function (ref, id) { | ||
return ( | ||
<ErrorBoundary fallback={<ErrorBoundaryContent />}> | ||
<GraphEditor ref={ref} id={id} /> | ||
</ErrorBoundary> | ||
); | ||
}; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NextJS was erroring because there's a circular dependency.
I think this is not too bad if we have some interface for global methods.
packages/graph-editor/src/components/panels/navigation/index.module.css
Outdated
Show resolved
Hide resolved
packages/graph-editor/src/components/panels/navigation/index.module.css
Outdated
Show resolved
Hide resolved
packages/graph-editor/src/components/panels/navigation/index.module.css
Outdated
Show resolved
Hide resolved
} | ||
|
||
.listItemCount { | ||
opacity: 0.5; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why the opacity here, to make it subtle?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, this should be solved with using var(--fg-subtle)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adds a panel with which the user can easily navigate between subgraphs.
Open it under Layout -> Navigator
2024-12-20-18-51_02.mp4