Differential D15032

Better fix for autocomplete blur (select on mousedown instead of delaying blur)
ClosedPublic
Actions

Authored by epriestley on Jan 15 2016, 5:40 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Dec 17, 1:29 PM

	Unknown Object (File)
	Mon, Dec 16, 3:49 AM

	Unknown Object (File)
	Sun, Dec 15, 2:47 PM

	Unknown Object (File)
	Fri, Dec 13, 4:38 PM

	Unknown Object (File)
	Sun, Dec 1, 11:00 AM

	Unknown Object (File)
	Tue, Nov 26, 2:26 AM

	Unknown Object (File)
	Sun, Nov 24, 1:00 AM

	Unknown Object (File)
	Thu, Nov 21, 9:34 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T10163: Remarkup Autocomplete Errata

Commits

Restricted Diffusion Commit
rP7f19216e4499: Better fix for autocomplete blur (select on mousedown instead of delaying blur)

Summary

Ref T10163. I would still sometimes not get a replacement after clicking with the delayed blur. This seems to fix the issue more consistently: instead of listening for a click event (which fires after the blur), listen for a mousedown event (which fires before the blur).

Test Plan

Observed consistent selection via mouse locally.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 36310.Jan 15 2016, 5:40 PM

epriestley retitled this revision from to Better fix for autocomplete blur (select on mousedown instead of delaying blur).

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

epriestley added a task: T10163: Remarkup Autocomplete Errata.

chad accepted this revision.Jan 15 2016, 5:47 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Jan 15 2016, 5:47 PM

Closed by commit rP7f19216e4499: Better fix for autocomplete blur (select on mousedown instead of delaying blur) (authored by epriestley, committed by epriestley). · Explain WhyJan 15 2016, 5:47 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

16 lines

webroot/

rsrc/

js/

phuix/

PHUIXAutocomplete.js

12 lines

Diff 36311

resources/celerity/map.php

Loading...

webroot/rsrc/js/phuix/PHUIXAutocomplete.js

Loading...