Homec4science

Fix an async display issue for tokenizer/typeahead results

Authored by epriestley <git@epriestley.com> on Feb 16 2014, 22:15.

Description

Fix an async display issue for tokenizer/typeahead results

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.

Reviewers: btrahan, chad

Reviewed By: chad

CC: aran

Maniphest Tasks: T4420

Differential Revision: https://secure.phabricator.com/D8250

Details

Committed
epriestley <git@epriestley.com>Feb 16 2014, 22:15
Pushed
aubortJan 31 2017, 17:16
Parents
rPH1c3373d77d9e: Add owner to task boards, tidy UI
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH340a293870c5: Fix an async display issue for tokenizer/typeahead results (authored by epriestley <git@epriestley.com>).Feb 16 2014, 22:15