Ticket #1175 (closed defect: no action needed)

Opened 9 months ago

Last modified 3 months ago

IE8 beta DomWalk broken/switch to TransformInnerHTML as default?

Reported by: ray Owned by:
Priority: normal Milestone: 0.96
Component: Browsers_InternetExplorer Version: trunk
Severity: major Keywords:
Cc:

Description

because getting node.attribute.length throws an "not implemented" error. This can only be a beta bug which we have to wait for being fixed. They can't mean this honestly!

Change History

Changed 9 months ago by ray

  • summary changed from IE8 beta DomWalk broken to IE8 beta DomWalk broken/switch to TransformInnerHTML as default?

rev [980]: set IE8 to use only TransformInnerHTML. Personally, I've always been kind of fond of the old method, but mybe this is the time to switch over to TransformInnerHTML as default? Wymsy, is this "original Xinha"?

Changed 8 months ago by wymsy

Not sure what you mean by "original Xinha", but ticket:253 documents the history. It was developed for Xinha specifically, if that's what you are asking.

Changed 8 months ago by gogo

My opinion is that DOMWalk should remain the default, because it is the "proper" way to do the job.

Using InnerHTML and a crapload of regexps is fine for speed when it works, but pretty rudely incomprehensible when it comes time to fix it :-)

As for attributes.length being unimplemented in IE8 (taking ray's word on that, I don't have IE8) it can only be a bug in IE8, and I don't think worth looking at it further.

Changed 4 months ago by gogo

See also #1250

Changed 3 months ago by ray

  • status changed from new to closed
  • resolution set to no action needed
Note: See TracTickets for help on using tickets.