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 comment:1
Changed September 16, 2006 03:41PM UTC by comment:2
| resolution: | → fixed |
|---|---|
| status: | new → closed |
Fixed in SVN.
Changed October 11, 2009 02:12PM UTC by 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: | reopened → closed |
| type: | → bug |
Reopened by spammer.
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.