Remove IE8 support that crashes in some cases (in my case using Drupa…

[joejoseph00]

Remove useless IE8 support (causes problems in some use cases, particularly when using jQuery ui with the Drupal contrib module named "views_load_more" module.
History is here: #1893
…l and the views_load_more module in combination with jQuery UI 12.1

[joejoseph00]

Yay, passing!

[mgol]

I set the milestone to 1.14.0 it as we agreed not to remove IE 8 workarounds from 1.13 yet. If you'd like to land it sooner, we'd need a way to have the patch apply only when needed. However, the existing check already tries to achieve exactly that... So it'd be good to get more details about the crash: what exactly causes the crash, in which browsers, etc.

[joejoseph00]

@mgol might as well wait for 1.14.0 .

with that said, IE8 is basically kaput for all intents and purposes. The only people running it would be those not connected to the internet and they'd probably not be changing versions of jQuery.

[linux-foundation-easycla]

The committers are authorized under a signed CLA.

• ✅ Joseph Olstad (3fbbfaf)

[joejoseph00]

I signed all the approvals now

[mgol]

Sure, no problem. Take note, though, that we've released 1.13.0 not so long ago so I wouldn't expect 1.14.0 to arrive any time soon. We want to let it stay on 1.13.0 for a while, let folks update, discover & fix regressions from 1.12.1, etc.

[mgol]

We'll be dropping all IE versions in 1.14.0 in a much larger PR. A part of it will be removing the $.fn._form API completely so this PR will not be needed anymore.

Thanks for your contribution anyway!

[mgol]

See https://blog.jqueryui.com/2024/04/plans-for-jquery-ui-1-14-dropping-support-for-ui-1-12-in-the-download-builder/ for more details.