jQuery: The Write Less, Do More JavaScript Library

Ticket #175 (reopened )

Opened 2 years ago

Last modified 2 years ago

XHTML Tables

Reported by: john Assigned to: anonymous
Type: Priority:
Milestone: Component:
Version: Keywords: ""
Cc: "" Needs:

Description

I just spotted in Revision 226, that jQuery will not by all means support XHTML as XML.

Line 743 reads:

if ( table && this.nodeName == "TABLE" && a[0].nodeName != "THEAD" ) {

nodeName in XML preserves case, that means in XHTML as XML it would return "table" and "thead" so that doesn't work here.

This may not have high priority but needs to be fixed, maybe in jQuery 1.01 or something?

Fix:

if ( table && this.nodeName.toUpperCase() == "TABLE" && a[0].nodeName.toUpperCase() != "THEAD" ) {

Attachments

Change History

Changed 2 years ago by MrBester

I would guess that this came from a non-IE direction as that returns uppercase regardless of the original case. That may be different in IE7, but I somehow doubt it. Either way, I heartily endorse this product and/or service.

Changed 2 years ago by joern

  • status changed from new to closed
  • resolution set to fixed

Fixed in SVN.

Note: See TracTickets for help on using tickets.