Page MenuHomePhabricator

Fix an async display issue for tokenizer/typeahead results
ClosedPublic

Authored by epriestley on Feb 16 2014, 7:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 29, 2:07 PM
Unknown Object (File)
Wed, Apr 24, 9:57 PM
Unknown Object (File)
Wed, Apr 24, 2:12 AM
Unknown Object (File)
Wed, Apr 24, 2:11 AM
Unknown Object (File)
Tue, Apr 23, 11:24 PM
Unknown Object (File)
Tue, Apr 23, 11:24 PM
Unknown Object (File)
Mon, Apr 22, 10:02 AM
Unknown Object (File)
Sun, Apr 21, 4:51 PM
Subscribers

Details

Summary

Ref T4420. After the changes to the tokenizer, I sometimes do this:

  • Type something like "diff" into a project typeahead.
  • Select "differential".
  • A fraction of a second later, the typeahead pops back open.

This is because I selected the result from a partial query (like "diff" running against the "di" results) and then the full results of the "diff" query came back to the browser.

Instead, when showing results, require that the current state match the state that the results are for: don't show "dog" results if the tokenizer now reads "cat", for whatever reason.

Test Plan

Added a 1s delay to results, typed "a", then typed "m" and selected a result in less than a second. Prior to the patch, the tokenizer would pop back open with "am" results afterward. Now, it doesn't.

Diff Detail

Repository
rP Phabricator
Branch
tokbeh
Lint
Lint Passed
Unit
No Test Coverage