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

Add support for smartcase? #109

Closed
briandipalma opened this issue Feb 17, 2023 · 3 comments
Closed

Add support for smartcase? #109

briandipalma opened this issue Feb 17, 2023 · 3 comments
Labels
duplicate This issue or pull request already exists

Comments

@briandipalma
Copy link

Basically if you type in an uppercase character make the search case sensitive?

@ggandor
Copy link
Owner

ggandor commented Feb 17, 2023

#28

@ggandor ggandor closed this as not planned Won't fix, can't repro, duplicate, stale Feb 17, 2023
@ggandor ggandor added the duplicate This issue or pull request already exists label Feb 17, 2023
@briandipalma
Copy link
Author

Interesting, I looked for "smart" but it didn't show up that issue.

Anyway, as you say yourself in #28:

It would work for the first input character

Plenty good enough for me, I'm constantly typing sT and then sighing as leap highlights matches for lower case ts. I've gone out of my way to type in an upper case letter as my first letter and yet leap ignores it. It seems perfectly reasonable that in such cases only upper case Ts should match. Doing that would often cut down the matches significantly and allow me to use easier to reach labels instead of more awkward ones.

@briandipalma
Copy link
Author

BTW I understand you say you "usually get a shorter route to the target" but I often find a single lower case match gets in the way of my uppercase match and if it wasn't there I could start editing immediately. So it's still a smoother flow if on the first character if it's uppercase the matches are smartcased.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants