Bug Tracker

Opened 4 years ago

Closed 4 years ago

#12777 closed bug (fixed)

Applets don't work when appended on IE

Reported by: Drawk Owned by: dmethvin
Priority: high Milestone: 1.9
Component: manipulation Version: 1.8.2
Keywords: Cc:
Blocked by: Blocking:


When I append an applet on IE 6, 7, 8 and sometimes 9 (only when mayscript is present as attribute on my test) the applet doesn't work.

If I change this (manipulation.js around the line 24) :

rnocache = /<(?:script|object|embed|option|style)/i,

by this :

rnocache = /<(?:script|object|embed|applet|option|style)/i,

the applet works.

Change History (6)

comment:1 Changed 4 years ago by dmethvin

  • Component changed from unfiled to manipulation
  • Milestone changed from None to 1.9
  • Priority changed from undecided to high

I can see how this could happen, even lacking a test case. Can you provide a simple test case using jsfiddle.net so we can be sure we're fixing the problem you've identified?

Most likely this will just go away if we stop using a fragment cache in 2.0, but I'm not sure if we'll be doing that refactoring for 1.9; if not we'll need to make that small change to not cache applets.

comment:2 Changed 4 years ago by dmethvin

  • Owner set to Drawk
  • Status changed from new to pending

comment:3 Changed 4 years ago by anonymous

Here is a test case that doesn't work for IE 6, 7 and 8 : http://jsfiddle.net/q6fm8/

comment:4 Changed 4 years ago by dmethvin

  • Status changed from pending to open

comment:5 Changed 4 years ago by dmethvin

  • Owner changed from Drawk to dmethvin
  • Status changed from open to assigned

comment:6 Changed 4 years ago by Dave Methvin

  • Resolution set to fixed
  • Status changed from assigned to closed

Fix #12777. Add applet to non-cacheable fragment types.

I don't want to add a unit test that creates a dependency on an applet.

Changeset: 02c726f71caf00a74968896b241ad206b1e1fc41

Note: See TracTickets for help on using tickets.