Page MenuHomePhabricator

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

Authored by epriestley on Jan 15 2016, 5:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 9:59 AM
Unknown Object (File)
Thu, May 2, 11:02 PM
Unknown Object (File)
Thu, Apr 25, 3:23 AM
Unknown Object (File)
Mon, Apr 22, 11:16 PM
Unknown Object (File)
Sun, Apr 21, 6:17 AM
Unknown Object (File)
Fri, Apr 19, 4:06 PM
Unknown Object (File)
Thu, Apr 11, 1:59 PM
Unknown Object (File)
Thu, Apr 11, 10:45 AM
Subscribers
None

Details

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
rP Phabricator
Branch
auto2
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 10145
Build 12307: Run Core Tests
Build 12306: arc lint + arc unit

Event Timeline

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.
chad edited edge metadata.
This revision is now accepted and ready to land.Jan 15 2016, 5:47 PM
This revision was automatically updated to reflect the committed changes.