Skip to content

Table with tfoot inside plain table results in weird DOM #3439

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
ShyLionTjmn opened this issue Dec 8, 2016 · 7 comments
Closed

Table with tfoot inside plain table results in weird DOM #3439

ShyLionTjmn opened this issue Dec 8, 2016 · 7 comments
Labels
Bug Manipulation

Comments

@ShyLionTjmn
Copy link

ShyLionTjmn commented Dec 8, 2016 •
edited
Loading

Create simple table with THEAD,TFOOT and TBODY tags and insert it into plain table with just TR and TD.
You will get broken table.

Here is an example:
https://jsfiddle.net/5kbwkzq8/5/
@timmywil
Copy link
Member

It seems like the same thing would happen without jQuery. We can't stop you from building invalid html. However, the test case is pretty large. Would it be possible to reduce this even further? There's a lot going there.

@gibson042
Copy link
Member

Ok, this one is kind of cool. A <tr> is being appended to a table that has no <tbody>, but contains an inner table that does have one. And when we try to find a <tbody> to contain the incoming row, we mistakenly use the one from the inner table. This should be a straightforward fix.

@gibson042 gibson042 added Bug and removed Needs info labels Dec 19, 2016
@timmywil
Copy link
Member

Thanks for digging into that!

@ShyLionTjmn
Copy link
Author

ShyLionTjmn commented Dec 20, 2016 •
edited
Loading

Why is it invalid? TBODY is not required when TR comes right after TABLE.

@timmywil
Copy link
Member

@ShyLionTjmn at first glance, it looked like an unclosed tag or something, but @gibson042 dug further and it is a jQuery bug.

gibson042 added a commit to gibson042/jquery that referenced this issue Dec 20, 2016
@gibson042
Manipulation: Restrict the tbody search to child nodes
1f614ed 
Fixes jquerygh-3439
@gibson042 gibson042 mentioned this issue Dec 20, 2016
Merged
@ShyLionTjmn
Copy link
Author

Ok. Thanks.

@gibson042 gibson042 mentioned this issue Dec 24, 2016
Closed
@leonardjo
Copy link

Seems indeed to be a duplicate of the issue I'm having. Commit 1f614ed fixes the issue I see. Thank you.

gibson042 added a commit that referenced this issue Jan 9, 2017
@gibson042
Manipulation: Restrict the tbody search to child nodes
efdb8a4 
For performance, use a querySelectorAll path instead of Javascript iteration.
http://codepen.io/anon/pen/vywJjx?editors=1010

Fixes gh-3439
Closes gh-3463
@lock lock bot locked as resolved and limited conversation to collaborators Jun 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Bug Manipulation
Milestone
No milestone
Development

No branches or pull requests
5 participants
@timmywil @leonardjo @markelog @gibson042 @ShyLionTjmn