Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 13 years ago

Closed 13 years ago

#26438 closed defect (bug) (fixed)

Theme Customizer is non-functional in IE11

Reported by:	celloexpressions	Owned by:	nacin
Priority:	highest omg bbq	Milestone:	3.8
Component:	Customize	Version:	3.8
Severity:	blocker	Keywords:	has-patch
Cc:		Focuses:

Description

Changing a control throws a JS error, and thus prevents the preview from updating and or the save button from becoming enabled.

The error is invalid calling object on line 265 of custommize-base.js.

Doesn't seem to affect Chrome or Firefox.

Introduced in [26548].

Attachments (1)

26438.patch (877 bytes ) - added by ocean90 13 years ago.

Download all attachments as: .zip

Change History (9)

#1 @lancewillett
13 years ago

Keywords needs-patch added
Milestone Awaiting Review → 3.8

#2 @nacin
13 years ago

Priority normal → highest omg bbq

We haven't updated Underscore.js since 1.4.4. I haven't seen anything in the logs to suggest that 1.5.2's version of _.isEqual would fix this.

#3 @nacin
13 years ago

At worst we can revert [26548] and save it for 3.9. I would rather not have that optimization in, if it causes problems. If it's caused at least one issue, there's bound to be another.

#4 @ocean90
13 years ago

Keywords has-patch added; needs-patch removed

I found https://github.com/paulmillr/exoskeleton/issues/36 and some other pages.

toString.call(a) should be replaced by Object.prototype.toString.call(a). This should be also reported upstream.
Patch attached.

@ocean90
13 years ago

Attachment 26438.patch added

#5 @ocean90
13 years ago

Seeing also an issue in IE 8 for hasOwnPropertery, see http://stackoverflow.com/questions/8157700/object-has-no-hasownproperty-method-i-e-its-undefined-ie8.

So +1 for a revert and review this again for 3.9, since the same code exists in 1.4.4 and us used by the media modal where _.isEqual it works.

#6 @nacin
13 years ago

Owner set to nacin
Resolution → fixed
Status new → closed

Customizer: Revert [26548], removing _.isEqual() for proper object comparison.

This broke the customizer in IE 11, with possibly other side effects. Revisit in 3.9.

Also reverts [26632].

see #26061 (reopens), fixes #26438.

#7 @EPinci
13 years ago

Resolution fixed
Status closed → reopened

Header image changing now works fine.
Changing header text color doesn't live preview as it does on Chrome.

#8 @ocean90
13 years ago

Resolution → fixed
Status reopened → closed

EPinci,
works for me in IE 11 with Twenty Fourteen.

Please open a new ticket if you think this is a bug in 3.8, but with some more Infos, like IE version, theme name and so on.

Re-closing since the reported issue was fixed.

Note: See TracTickets for help on using tickets.

Download in other formats: