507853 - <link> within <tr> causes unexpected spacing

Status: RESOLVED FIXED

(Core :: DOM: HTML Parser, defect)

Description

[Vicky]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1.2) Gecko/20090729 Firefox/3.5.2 GTB5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1.2) Gecko/20090729 Firefox/3.5.2 GTB5

When ever I visit Mumbai university site Through Mozilla Firefox 3 Then i Web Homepage of mumbai university Menu  section  only seems..... in Below Blank  Then .. if We scroll Mouse Left to right then content of Home page ..we able to see, But in Internet Explorer, Evn Mozilla Firefox below version seems properly ...  ..So please chk fix the bug.
Thanks

Reproducible: Always

Comment 1

[Michael Ryan]

Attachment: reduced testcase

Here's a reduced testcase from the tag soup. Other browsers I've tested have the second "foo" vertically-aligned with the first.

Comment 2

[Masatoshi Kimura [:emk]]

HTML5 parser fixed the issue.

Comment 3

[Michael Ryan]

The HTML5 parser changes how it generates the DOM, but the spacing issue still exists if the node is dynamically added:

javascript:document.getElementsByTagName('tr')[1].insertBefore(document.createElement('link'), document.getElementsByTagName('tr')[1].firstChild);void(0);

I guess it might be better to change the testcase to do this...

Comment 4

[Michael Ryan]

Attachment: reduced testcase for HTML5 parser

This testcase demonstrates the issue even with the HTML5 parser is enabled.

Comment 5

[Michael Ryan]

So I think there are three different issues here:
1) What should the parser do with a <link> in a <tr>? The HTML5 parser deals with this in a reasonable manner.
2) Should there be a default "link { display: none; }" rule?
3) What should be done with non-<td> element children of a <tr>?

I'd imagine that all three probably have existing bugs or have been discussed before - just need to find out where.

Comment 6

[Henri Sivonen (:hsivonen)]

The HTML5 parser fixes the parser bug. Filed bug 563884 about the layout side.