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
F19060568: D8250.id.diff
Sat, Nov 29, 9:41 AM
F19059326: D8250.id.diff
Sat, Nov 29, 5:10 AM
F19052172: D8250.diff
Fri, Nov 28, 3:27 AM
F19049635: D8250.id19630.diff
Thu, Nov 27, 5:51 PM
F19031421: D8250.diff
Tue, Nov 25, 1:03 AM
F18856050: D8250.diff
Sat, Nov 1, 7:44 AM
F18837453: D8250.id.diff
Oct 27 2025, 5:30 AM
F18836412: D8250.id19630.diff
Oct 27 2025, 12:51 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