Skip to content
This repository has been archived by the owner on Jun 5, 2023. It is now read-only.

Search should consider consecutive letters only #4

Closed
damithc opened this issue Jun 27, 2017 · 3 comments
Closed

Search should consider consecutive letters only #4

damithc opened this issue Jun 27, 2017 · 3 comments
Labels

Comments

@damithc
Copy link

damithc commented Jun 27, 2017

e.g.
med should not match mergeApproved
image

@damithc damithc added the p.Low label Jun 27, 2017
@jiajunGit
Copy link
Collaborator

jiajunGit commented Jun 30, 2017

@damithc I have modified the search algo in commit b1fdeab to consider only consecutive characters but it is more forgiving towards characters like . or - or spaces so if you type aallocated, it will match a-allocated

I will keep this issue open because I want to see if this search algo is what you have in mind. If this is the search algo you want, you can close the issue. If not, you can state some modifications you want to be made to the search algo and I will try to alter it.

@damithc
Copy link
Author

damithc commented Jun 30, 2017

I will keep this issue open because I want to see if this search algo is what you have in mind. If this is the search algo you want, you can close the issue. If not, you can state some modifications you want to be made to the search algo and I will try to alter it.

Roger that 👍

@damithc
Copy link
Author

damithc commented Jul 4, 2017

The current behavior is good. Closing for now. Will reopen if something comes up.

@damithc damithc closed this as completed Jul 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants