Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 10 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:

Description

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 10 years ago.

Download all attachments as: .zip

Change History (4)

@azaozz
10 years ago

#1 @azaozz
10 years ago

  • Keywords has-patch added

#2 @azaozz
10 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
10 years ago

Nice! Easy win.

Note: See TracTickets for help on using tickets.