]> source.dussan.org Git - jquery.git/commit
Event: Simulate focus/blur in IE via focusin/focusout (3.x version)
authorMichał Gołębiowski-Owczarek <m.goleb@gmail.com>
Mon, 27 Mar 2023 19:24:42 +0000 (21:24 +0200)
committerGitHub <noreply@github.com>
Mon, 27 Mar 2023 19:24:42 +0000 (21:24 +0200)
commit59f7b55bf6dad71eee6434116566356d597f1d90
treedc6940dbcb6afc293ef659ef9b6894ed6e3857cd
parent4837a95b367faa613105a03fa0cb8b8fc50c02d4
Event: Simulate focus/blur in IE via focusin/focusout (3.x version)

In IE (all versions), `focus` & `blur` handlers are fired asynchronously
but `focusin` & `focusout` are run synchronously. In other browsers, all
those handlers are fired synchronously. Asynchronous behavior of these
handlers in IE caused issues for IE (gh-4856, gh-4859).

We now simulate `focus` via `focusin` & `blur` via `focusout` in IE to avoid
these issues. This also let us simplify some tests.

This commit also simplifies `leverageNative` - with IE now using `focusin`
to simulate `focus` and `focusout` to simulate `blur`, we don't have to deal
with async events in `leverageNative`. This also fixes broken `focus` triggers
after first triggering it on a hidden element - previously, `leverageNative`
assumed that the native `focus` handler not firing after calling the native
`focus` method meant it would be handled later, asynchronously, which
was not the case (gh-4950).

To preserve relative `focusin`/`focus` & `focusout`/`blur` event order
guaranteed on the 3.x branch, attach a single handler for both events in IE.

A side effect of this is that to reduce size the `event/focusin` module
no longer exists and it's impossible to disable the `focusin` patch
in modern browsers via the jQuery custom build system.

Fixes gh-4856
Fixes gh-4859
Fixes gh-4950
Ref gh-5223
Closes gh-5224

Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
README.md
src/event.js
src/event/focusin.js [deleted file]
src/event/support.js [deleted file]
src/jquery.js
test/unit/event.js
test/unit/support.js