WordPress.org
Get WordPress

Make WordPress Core

Opened 5 years ago

Closed 2 years ago

#48523 closed defect (bug) (wontfix)

jquery.js parse error in IE8

Reported by: jqz's profile jqz Owned by:
Milestone: Priority: normal
Severity: normal Version: 5.2.1
Component: External Libraries Keywords:
Focuses: javascript Cc:

Description (last modified by SergeyBiryukov)

This is a follow-up to #47020.

The minification of jquery.js in [45342] is not compatible with IE<=8.

The error is:

SCRIPT1028: Expected identifier, string or number
jquery.js (4,13727)

It's caused by

propFix:{for:"htmlFor",class:"className"}

because for and class, being reserved words, need to be in quotes.

I know that WordPress 5.2.x no longer supports IE8 (see #47761). However the change has been ported back as far as WordPress 4.5.x ([46027]), which I'm pretty sure is supposed to support IE8.

Change History (5)

#2 @SergeyBiryukov
5 years ago

  • Description modified (diff)
  • Focuses javascript added

#3 @SergeyBiryukov
5 years ago

  • Version changed from trunk to 5.2.1

#4 @Hareesh Pillai
5 years ago

  • Keywords needs-patch added

#5 @desrosj
2 years ago

  • Keywords needs-patch removed
  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Thanks for this one @jqz. Apologies that this one fell through the cracks.

A lot has changed in 3+ year, and even IE11 is now unsupported by Microsoft.

I'm going to close this one out as the time to fix this has passed, and with IE8 usage so low (​0.015%), it's not something that's a good use of time.

Note: See TracTickets for help on using tickets.