Use smartcase for vomnibar queries. #721

Merged
merged 3 commits into from Dec 16, 2012

Projects

None yet

2 participants

@smblott-github
Collaborator

It's probably obvious, but ... Why would we want this?

  • Smartcase is the best of both worlds.
  • We already have smartcase for "Find" in the front end, it's inconsistent not to have it for the vomnibar too.

This PR also:

  • Adjusts existing unit tests for smartcase.
  • Adds tests for smartcase.
  • Adds tests for queries including RegExp meta-characters.
@int3 int3 merged commit 6eca29f into philc:master Dec 16, 2012

1 check passed

default The Travis build passed
Details
@int3
Collaborator
int3 commented Dec 16, 2012

Thanks @smblott-github! I'll get to the rest of your PRs over the next few weeks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment