-
Notifications
You must be signed in to change notification settings - Fork 31
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
Multi word search #19
Comments
@troyd1000 thanks for the time spent with this enhancement, but I'm afraid this is not the correct way to contribute. Opening a zip file is risky, therefore I'm not doing it. If you wish to contribute, please fork the project, add your code and then create a pull request, please. |
Jonatas, sorry, all I have ever done with github is download code and did not know how that worked. I think I worked through adding a fork, updating the code and adding a pull request. Let me know if it worked.
|
It did work!! =) I'll take a look at it and run some tests as soon as possible. |
@troyd1000 , you have modified the wrong file, the transpiled javascript file. You have to modify the Typescript file. |
I don’t know how to do typescript and don’t have the time to figure it out right now. If you wish to interpret what I did and add it to your project feel free. Otherwise, you can just let it go. I have spent a lot of time on figuring out updating things on github and don’t have time to do more.
|
Ok, no problem. And thanks again! I'll keep this issue open in case anyone wants to pick up from where you left. |
One thing it seems no one has and that is also with jquery autocomplete is multi word search. Say in your example, if I do bo au ex, it would find anything matching all those words. It is a nice way to whittle down a list matching multiple things. I made updates to your JS to do this. Feel free to implement them to your core if you like.
attached is the updated file.
ac.zip
The text was updated successfully, but these errors were encountered: