[wp-trac] [WordPress Trac] #48523: jquery.js parse error in IE8

[WordPress Trac]

#48523: jquery.js parse error in IE8
--------------------------------+------------------------------
 Reporter:  jqz                 |       Owner:  (none)
     Type:  defect (bug)        |      Status:  new
 Priority:  normal              |   Milestone:  Awaiting Review
Component:  External Libraries  |     Version:  trunk
 Severity:  normal              |  Resolution:
 Keywords:                      |     Focuses:  javascript
--------------------------------+------------------------------
Changes (by SergeyBiryukov):

 * focuses:   => javascript


Old description:

> This is a follow-up to #47020.
>
> The minification of `jquery.js` in
> https://core.trac.wordpress.org/changeset/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
> (https://core.trac.wordpress.org/changeset/46027), which I'm pretty sure
> **is** supposed to support IE8.

New description:

 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.

--

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/48523#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform