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

Add id search to dataflow viewer #1379

Merged
merged 3 commits into from
Nov 29, 2023
Merged

Conversation

mcnuttandrew
Copy link
Contributor

@mcnuttandrew mcnuttandrew commented Nov 25, 2023

Screenshot 2023-11-25 at 4 55 52 PM

When debugging things in the vega data flow graph sometimes it can be tedious to have to manually search for a particular node of interest. This adds a very simple id search which allows the user to search for a particular node

@@ -198,7 +198,7 @@ class Editor extends React.PureComponent<Props> {
prevProps.editorRef && prevProps.editorRef.deltaDecorations(prevProps.decorations, []);
}

if (this.props.parse) {
if (this.editor && this.props.parse) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure what this is, but i couldn't get the editor to run local without it?

Copy link
Member

@domoritz domoritz left a comment

Choose a reason for hiding this comment

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

Thanks for the feature. Let's make this look a bit nicer. Let's all caps ID. Maybe we can add some spacing to the button, make it inline with the input (IDs are short), make the button full width like the input, or even remove the button and search immediately on change. Also, should we use type search (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/search)?

Screenshot 2023-11-26 at 11 37 23

@mcnuttandrew
Copy link
Contributor Author

Thanks for this thoughtful feedback. I believe i've addressed all of it, you can see a preview here:

Screenshot 2023-11-28 at 5 41 34 PM

or even remove the button and search immediately on change

I found this to be a little bit jerky when i was trying it out, but happy to change over if you feel strongly

@domoritz domoritz merged commit 53bc4d3 into vega:master Nov 29, 2023
1 check passed
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