Bug Tracker

Modify ↓

Ticket #175 (closed bug: fixed)

Opened 7 years ago

Last modified 4 years ago

XHTML Tables

Reported by: john Owned by:
Priority: undecided Milestone:
Component: ajax Version:
Keywords: "" Cc: ""
Blocking: Blocked by:

Description (last modified by dmethvin) (diff)

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" ) {

Change History

comment:1 Changed 7 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.

comment:2 Changed 7 years ago by joern

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

Fixed in SVN.

comment:4 Changed 4 years ago by dmethvin

  • Status changed from reopened to closed
  • Description modified (diff)
  • Type set to bug
  • Component set to ajax
  • Priority set to blocker
  • need set to Review
  • Resolution set to fixed

Reopened by spammer.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View ↑

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.