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

Added function to tree widget to find a node by the data field #5362

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

BurnoutDV
Copy link

Based on the monologue discussion #5315

I added a function to the Tree Widget for finding a node based on the supplied data field. Its probably inefficient but I couldn't think of a better way to achieve what I wanted with arbitrary json data. There is also the case that there might be a list instead of a dictionary where one might want to find a node that matches any one value.

I could extend the functionality to find all matching nodes and make it match any one value in a list or nested list if there is interest for the basic idea.

Maybe finding notes based on supplied data is to much of a deviation from the base implementation.

Please review the following checklist.

  • [ X] Docstrings on all new or modified functions / classes
  • [ ] Updated documentation
  • [ ] Updated CHANGELOG.md (where appropriate)

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