Skip to content

Conversation

@danielnilsson9
Copy link
Contributor

#96

@danielnilsson9
Copy link
Contributor Author

I guess a more generic solution to this would be to make keybindings conditional as you suggested in PR #76 so I'm not sure if it is a good idea to add this if it will be obsolete later?

@jameskerr
Copy link
Contributor

I think this is great. Even though we'll add keyboard shortcuts, there may be some nodes that you don't want to make editable. Thank you for adding to the demo as well.

@jameskerr jameskerr merged commit ec40e35 into brimdata:main Jan 23, 2023
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