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)
Wed, Apr 10, 11:38 PM
Unknown Object (File)
Sat, Mar 30, 10:58 PM
Unknown Object (File)
Feb 18 2024, 8:43 AM
Unknown Object (File)
Feb 4 2024, 3:01 AM
Unknown Object (File)
Jan 27 2024, 9:09 AM
Unknown Object (File)
Jan 6 2024, 11:16 PM
Unknown Object (File)
Dec 29 2023, 2:45 PM
Unknown Object (File)
Dec 23 2023, 9:22 AM
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

Lint
Lint Skipped
Unit
Tests Skipped