-
Notifications
You must be signed in to change notification settings - Fork 266
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
Search seems to only search in the first node #111
Comments
If it is indeed an issue, I'll see if I can help, now that the GDPR hype is over I have more time.. |
Yeah I noticed it a few times but couldn't put my finger on it. I think it may have to do with the accented characters. We could use unicode aware regex search instead but that would mean IE 11 and below users will be out of luck (we could use something like regexpu but I don't want to add bloat for obsolete browsers). |
Ill try by removing all occurences of åäö later today |
Adding support for that seems Overkill |
If there is fair demand I don't mind supporting unicode characters. IE bloat is something I'd like to avoid at all costs. |
FWIW, I'm running into this issue as well, and from what I can tell I don't think it appears to be related to special characters (I don't have these characters in my data set). It looks like search is only searching the first X # of nodes. For example in the large-tree example, the cutoff is around node "enbacken123-eneby-d3". If you search for anything after that you will get no results. In my local project I see search also only getting results from roughly the first X # of nodes. In my local project I've tried going back to 1.10.0 to test, and I do see all matches there, but of course this is before the search enhancements, so performance is too slow to be usable (I'm looking to support ~15k nodes as well). |
@zig145 Any chance you can share your dataset? Random/redacted values would do as long as they are representative of the tree's complexity. |
I'm dealing with sensitive data, so I've rewritten the names as a simple key for this example. When loading this data into the component and searching, I don't ever see results after the "-0-2-0" node. |
@zig145 I think this will do.
Now I know what to look for. Thanks for the tip! |
I found the problem. But it's not an easy one to fix. :( |
🎉 This issue has been resolved in version 1.11.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
praise be mrcheif |
Still happening! version Try to search for "gala" in your example https://dowjones.github.io/react-dropdown-tree-select/#/story/large-tree |
@fpauer can you try the beta version and let me know? |
@mrchief now it is working, version |
https://dowjones.github.io/react-dropdown-tree-select/#/story/large-tree
When I try, it seems to only find results from the first node. Maybe I'm doing something wrong. Tried with my own data, had the exact same behaviour.
The text was updated successfully, but these errors were encountered: