Opened 9 years ago
Closed 9 years ago
#36793 closed defect (bug) (fixed)
Customizer doesn't load in IE8
Reported by: | ocean90 | Owned by: | westonruter |
---|---|---|---|
Milestone: | 4.5.3 | Priority: | normal |
Severity: | normal | Version: | 4.5 |
Component: | Customize | Keywords: | has-patch commit fixed-major |
Focuses: | javascript | Cc: |
Description
Error message:
Webpage error details User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; WOW64; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0) Timestamp: Mon, 9 May 2016 20:28:34 UTC Message: Expected identifier Line: 1454 Char: 7 Code: 0 URI: http://develop.wp.dev/wp-includes/js/underscore.js?ver=1.8.3
Attachments (2)
Change History (14)
#3
@
9 years ago
We need to get the QUnit tests running on BrowserStack so that these IE8 issues can be caught automatically.
#4
@
9 years ago
Digging into this, so far I have discovered this is theme related. Activating an older theme the customizer loads without error in ie8.
#5
@
9 years ago
- Keywords has-patch commit added; needs-patch removed
- Owner set to westonruter
- Status changed from new to assigned
In 36793.diff:
- Remove use of reserved word
default
in Underscore template which breaks IE8.
This code was introduced in r30712 and is inclass-wp-customize-media-control.php
- not sure why this cropped up in 4.5 specifically, perhaps something in the customizer load sequence is now using the media control (site logo?) that wasn't previously being used.
This change fixes the issue. Screenshot showing no errors in customizer in IE8:
#7
@
9 years ago
- Keywords fixed-major added
- Milestone changed from Awaiting Review to 4.5.3
- Resolution fixed deleted
- Status changed from closed to reopened
Re-opening for consideration for 4.5.3
#9
follow-up:
↓ 10
@
9 years ago
@ocean90 aha that makes more sense. I was using Svn blame, how did you find the breaking change?
#10
in reply to:
↑ 9
@
9 years ago
Replying to adamsilverstein:
SVN blame :-)
Go to https://core.trac.wordpress.org/browser/trunk/src/wp-includes/customize/class-wp-customize-media-control.php?annotate=blame&rev=37152&marks=217#L217 and click on the changeset number in the first column.
Previously: #30781