Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#31276 closed enhancement (fixed)

Load json2.js only in browsers that need it

Reported by: azaozz's profile azaozz Owned by: azaozz's profile azaozz
Milestone: 4.2 Priority: normal
Severity: normal Version:
Component: Script Loader Keywords: has-patch
Focuses: Cc:


Now that we have IE conditional comments for JS, we should be loading json2.js only in browsers that still need it, lt IE 8. That would load about 18kb less JS.

Attachments (1)

31267.patch (815 bytes) - added by azaozz 9 years ago.

Download all attachments as: .zip

Change History (4)

9 years ago

#1 @azaozz
9 years ago

  • Keywords has-patch added

#2 @azaozz
9 years ago

  • Owner set to azaozz
  • Resolution set to fixed
  • Status changed from new to closed

In 31526:

Load json2.js only in IE7 and older. Fixes #31276.

#3 @nacin
9 years ago

Nice! Easy win.

Note: See TracTickets for help on using tickets.