-
Notifications
You must be signed in to change notification settings - Fork 22
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
Exact Search Match should be easier to find #76
Comments
yeaaaaaaaaaa! |
Implementation idea: while looping, add a re.match (flag ignore case) to check if an exact match is found. Add those to beggining of list instead of appending. On front/js side, when creating DOM elements, check if query matches heading, if it does, apply a css class that will make those items stand out at the top Ensure there is not significant performance decrease by running the search benchmark test |
Exact match is raise and "bold" |
YEAAAAA!
Nice man! So helpful :D
…On Thu, Feb 9, 2017 at 9:28 PM, Gui Talarico ***@***.***> wrote:
Exact match is raise and "bold"
[image: image]
<https://cloud.githubusercontent.com/assets/9513968/22811743/b517be84-ef0e-11e6-91dc-8c6505adccd4.png>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#76 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/APGhajaGS9Oq2VWyGsbpZ_C4hvbMlhqIks5ra8tbgaJpZM4L8Xdu>
.
--
*WeWork | Brian Lee*
Buildings Research
Product Strategy R&D
wework.com <http://www.wework.com/>
Create Your Life's Work
|
If a user search
Wall Class
search will returnFace Wall
StructuralWall
and many others that include the wordsWall
andClass
.The entry that matches the query 100%, perhaps should be given a higher priority and pushed to the top.
(especially considering the query might have been assisted by autocomplete, it means there is a high chance that the exact match is what the user was looking for.
The text was updated successfully, but these errors were encountered: