Autocomplete: Removed the timeout for the change event. Fixes #7550 #596
Conversation
… Autocomplete change event not triggered in time. Thanks spekary for finding a workaround for IE.
|
@spekary I did some manual testing in Chrome and IE and I'm getting the change event before the form's submit event now. Can you confirm? |
|
This looks good in my tests as well. Everything seems to be functioning correctly, including the test in the bug report. I may not have thought of every scenario but all of the demos seem fine and playing around with forms and submitting and making sure events fire in the correct order all seem good to me. |
|
Looks good here too. Tested on Mac Safari, Mac Netscape, IE 7 in Win XP, IE 9 in Win7. Checked both this problem and the other problem I mentioned regarding canceling the menu when you tab out during an async lookup. Was concerned that you took out cancelSearch = true in the blur, but since its closing immediately now, its all good. Looking forward to getting it stable. |
|
Landed in f5b52a8. |
This is a duplicate of PR 508, but rebased and squashed.
I'd like at least one other person to review this before we land it, just to make sure I didn't miss any scenarios.