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

[CLOSED] StringMatch should boost matches with uppercase letters in query #3067

Open
core-ai-bot opened this issue Aug 29, 2021 · 6 comments

Comments

@core-ai-bot
Copy link
Member

Issue by njx
Wednesday Mar 27, 2013 at 16:17 GMT
Originally opened as adobe/brackets#3263


I was using Quick Open to open "Package.js", and got a lot of higher-ranked hits for "package.json". This is probably not a common case, but it occurred to me that it might be a good idea (and relatively harmless to other use cases) if we give a score boost to an uppercase letter in the query string that matches an uppercase letter in the candidate string (vs. a lowercase letter in the candidate), since if you specifically typed it in uppercase in the query, you probably know it's uppercase in the name you want. (This would have the added bonus of giving an easy way to boost acronym/camel-case searches.)

@core-ai-bot
Copy link
Member Author

Comment by njx
Wednesday Mar 27, 2013 at 16:17 GMT


Low pri to@dangoor

@core-ai-bot
Copy link
Member Author

Comment by njx
Monday Apr 01, 2013 at 18:04 GMT


Reviewed.

@core-ai-bot
Copy link
Member Author

Comment by dangoor
Monday Mar 17, 2014 at 19:48 GMT


This appears to basically be the same as #3971 and #3694

@core-ai-bot
Copy link
Member Author

Comment by njx
Friday Apr 11, 2014 at 16:16 GMT


Removing 1.0 milestone - if this does happen to get fixed by the same fix as #2663 (which is still on the 1.0 list) that's cool, but we don't think this one is necessary for 1.0 by itself.

@core-ai-bot
Copy link
Member Author

Comment by JeffryBooher
Tuesday Oct 21, 2014 at 21:42 GMT


@dangoor I followed the steps@njx posted in this issue and #9621 doesn't seem to fix it. The tests I was running worked but now I don't know if they just "worked" for other reasons. Maybe I'm doing it wrong but, when I type Package.js, I get package.json followed by Package.js in the QuickOpen list.

@core-ai-bot
Copy link
Member Author

Comment by JeffryBooher
Wednesday Oct 22, 2014 at 14:36 GMT


Closing as fixed.

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

No branches or pull requests

1 participant