Skip to main content

Bug Tracker

Side navigation

#175 closed bug (fixed)

Opened September 02, 2006 10:26AM UTC

Closed October 11, 2009 02:12PM UTC

XHTML Tables

Reported by: john Owned by:
Priority: undecided Milestone:
Component: ajax Version:
Keywords: "" Cc: ""
Blocked by: Blocking:
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 (0)
Change History (3)

Changed September 13, 2006 11:55AM UTC by MrBester comment:1

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 September 16, 2006 03:41PM UTC by joern comment:2

resolution: → fixed
status: newclosed

Fixed in SVN.

Changed October 11, 2009 02:12PM UTC by dmethvin comment:3

component: → ajax
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" ) {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" ) {
need: → Review
priority: → blocker
resolution: → fixed
status: reopenedclosed
type: → bug

Reopened by spammer.