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

Search seems to only search in the first node #111

Closed
josvegit opened this issue May 31, 2018 · 15 comments
Closed

Search seems to only search in the first node #111

josvegit opened this issue May 31, 2018 · 15 comments
Labels

Comments

@josvegit
Copy link

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.

@josvegit
Copy link
Author

If it is indeed an issue, I'll see if I can help, now that the GDPR hype is over I have more time..

@mrchief
Copy link
Collaborator

mrchief commented May 31, 2018

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).

@josvegit
Copy link
Author

Ill try by removing all occurences of åäö later today

@josvegit
Copy link
Author

Adding support for that seems Overkill

@mrchief
Copy link
Collaborator

mrchief commented May 31, 2018

If there is fair demand I don't mind supporting unicode characters. IE bloat is something I'd like to avoid at all costs.

@zig145
Copy link

zig145 commented Jun 4, 2018

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).

@mrchief
Copy link
Collaborator

mrchief commented Jun 4, 2018

@zig145 Any chance you can share your dataset? Random/redacted values would do as long as they are representative of the tree's complexity.

@zig145
Copy link

zig145 commented Jun 4, 2018

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.

myData.zip

@mrchief
Copy link
Collaborator

mrchief commented Jun 4, 2018

@zig145 I think this will do.

I don't ever see results after the "-0-2-0" node.

Now I know what to look for. Thanks for the tip!

@mrchief mrchief added the bug label Jun 4, 2018
@mrchief
Copy link
Collaborator

mrchief commented Jun 6, 2018

I found the problem. But it's not an easy one to fix. :(

@mrchief
Copy link
Collaborator

mrchief commented Jun 6, 2018

🎉 This issue has been resolved in version 1.11.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@josvegit
Copy link
Author

josvegit commented Jun 8, 2018

praise be mrcheif

@fpauer
Copy link

fpauer commented Oct 16, 2018

Still happening! version 1.12.7

Try to search for "gala" in your example https://dowjones.github.io/react-dropdown-tree-select/#/story/large-tree

image_uploaded_from_ios

@mrchief
Copy link
Collaborator

mrchief commented Oct 17, 2018

@fpauer can you try the beta version and let me know?

@fpauer
Copy link

fpauer commented Nov 29, 2018

@mrchief now it is working, version 1.13.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants