Make "Locate File" trigger when data loads if the user typed/pasted real fast
ClosedPublic
Actions

Authored by epriestley on Apr 19 2017, 11:12 PM.

Tags

None

Referenced Files

	F18679162: D17739.id.diff
	Fri, Sep 26, 12:33 AM

	F18624199: D17739.id.diff
	Sep 15 2025, 7:16 PM

	F18586267: D17739.diff
	Sep 11 2025, 6:03 PM

	F18468209: D17739.diff
	Sep 2 2025, 2:41 PM

	F18429798: D17739.id.diff
	Aug 31 2025, 3:19 AM

	F18418090: D17739.diff
	Aug 30 2025, 12:40 PM

	F18113507: D17739.id.diff
	Aug 13 2025, 2:26 AM

	F18110346: D17739.id42669.diff
	Aug 11 2025, 6:50 PM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Maniphest Tasks

T12599: Diffusion "Locate File" typeahead fails to trigger when pasting a filename into the input field

Commits

rPd2560b246201: Make "Locate File" trigger when data loads if the user typed/pasted real fast

Summary

Fixes T12599. If you're faster than the network request, we don't actually resolve your query when the data arrives.

Instead, when the data shows up, run the query if the user has typed something.

Test Plan

Pasted a filename in real fast, got results. (Previously: no results until you press something else.)

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 19 2017, 11:12 PM

Harbormaster completed remote builds in B16604: Diff 42669.Apr 19 2017, 11:13 PM

👍

This revision is now accepted and ready to land.Apr 19 2017, 11:13 PM

Closed by commit rPd2560b246201: Make "Locate File" trigger when data loads if the user typed/pasted real fast (authored by epriestley, committed by epriestley). · Explain WhyApr 19 2017, 11:14 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

16 lines

webroot/

rsrc/

js/

application/

diffusion/

DiffusionLocateFileSource.js

5 lines

Diff 42670

View Options

resources/celerity/map.php