Page MenuHomePhabricator

Improve typeahead behavior with mixed-case matches
ClosedPublic

Authored by epriestley on Nov 8 2016, 7:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Feb 18, 12:00 PM
Unknown Object (File)
Sun, Feb 9, 2:45 AM
Unknown Object (File)
Tue, Feb 4, 2:21 AM
Unknown Object (File)
Mon, Jan 27, 1:22 PM
Unknown Object (File)
Sun, Jan 26, 11:54 PM
Unknown Object (File)
Sat, Jan 25, 12:15 AM
Unknown Object (File)
Fri, Jan 24, 5:36 PM
Unknown Object (File)
Jan 21 2025, 9:53 AM
Subscribers
None

Details

Summary

Ref T8510. We had two issues with mixed-case result sorting, like typing @joe to match user Joe.

  • The fallback sort was not normalized properly, so "J" could sort after "j". Instead, normalize values for sorting.
  • The prefix_hits and older priority_hits mechanisms were competing destructively. The prefix_hits mechanism completely replaces the priority_hits mechanism. Instead, use only the prefix_hits mechanism.
Test Plan
  • Copied results for "joe" from WMF.
  • Hard-coded the controller to return them.
  • Searched for @joe.
  • After patches, first hit is user "Joe".

Diff Detail

Repository
rP Phabricator
Branch
typeahead1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 14407
Build 18767: Run Core Tests
Build 18766: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Improve typeahead behavior with mixed-case matches.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 8 2016, 7:41 PM
This revision was automatically updated to reflect the committed changes.