Closed Bug 1262 Opened 26 years ago Closed 25 years ago

wrong layout for empty tables

Categories

(Core :: DOM: HTML Parser, defect, P2)

Product:
Core
Component:
DOM: HTML Parser
Platform:
x86
Windows NT
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: buster, Assigned: harishd)

References

Details

(Keywords: testcase, Whiteboard: Table should be a single pixel line all across the horizontal width of the window)

Attachments

(1 file)

This is the test described in this bug
264 bytes, text/html
Details 
<html> <body>
 1
<table border=1 width="100%">
<tr><td></td></tr>
</table>
2
<br>
3
<table border=1 width="100%">
<tr></tr>
</table>
4
</body> </html>
Status: NEW → ASSIGNED
Assignee: buster → rickg
Status: ASSIGNED → NEW
Component: HTMLTables → Parser
The table code is doing the right thing with the content that it is being
given.  The "problem" is with the parser (or content sink?)
I think what raptor does is compliant with the HTML 4.0 DTD.  Perhaps this
should be a "quirk"?  The quirk behavior would be to throw away any table that
didn't have at least one cell.  Quite a bit of experimenting would be necessary
to determine exactly what cases would need to be addressed.
Assignee: rickg → buster
hang on there bucko. This is a not a parser bug. You should address this issue
with Kipp.
Status: NEW → ASSIGNED
Assignee: buster → karnaze
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
Setting all current Open/Normal to M4.
per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
Target Milestone: M4 → M6
moving to M6
Moving to M8.
This behavior looks nearly correct to me, as since a table is there, but there
is nothing in it, no </td>'s and the only <tr> is empty, with only a border, so
I think showing a 1 pixel dot like that is technically correct, but since the
table width was set at 100%, should the pixel thing not be a line across the
screen? Since there is only one row, but the table has no depth, I do think that
a 1 pixel wide line would be right to draw with that code. This may be realted
to bug#5797 , which is causing a tables width not to be displayed properly. What
it does is that it only makes it as wide as is needed to fit text, pictures etc.
But since this is empty no room is needed and it only dispalyes the dot to
satisfy the border=1 atribute?
Moving to M9.
Whiteboard: [MAKINGTEST] jonesev@home.com July 28
Whiteboard: [MAKINGTEST] jonesev@home.com July 28 → [TESTCASE] Table should be a single pixel line all across the horizontal width of the window
I attached the HTML file mentioned by buster@netscape.com and added one
additional bit:

<br>
5
<table border=1 width="100%">
<tr><td>&nbsp;</td></tr>
</table>
6

As of the July 26 Linux build, the top table is close to correct (should be a
single pixel line, but instead it has a space in there). The middle box is
incorrect (should it be a line or not there at all because it is an incorrect
table?) and the bottom box is close (It is correct but with a thicker than one
pixel border. The same behavior is evident in the Win32 version.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
The layout is now very close to Nav4.x except that the 3 and 4 appear on
different lines, which is what IE5 does. The first table can be made less tall
by setting cellspacing and cellpadding to 0. If someone can find in any spec
support for the notion that the 1st table should be a single line, I'll make it
work that way in Standard mode.
Status: RESOLVED → VERIFIED
verified 1999080508
Status: VERIFIED → REOPENED
The layout is wrong again, using 1999102208 build.
Reopening.
Assignee: karnaze → harishd
Status: REOPENED → NEW
Harish, a content dump reveals that the lines with "1" and "2" are coming before
the 1st table which is a regression.
Status: NEW → ASSIGNED
Aware of the problem and I've the fix in hand.
Resolution: FIXED → ---
Target Milestone: M9
*** Bug 17113 has been marked as a duplicate of this bug. ***
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → FIXED
Rectified the problem in CNavDTD::AddLeaf(). Marking FIXED.
Status: RESOLVED → VERIFIED
verified again
URL: see test case
Keywords: testcase
Whiteboard: [TESTCASE] Table should be a single pixel line all across the horizontal width of the window → Table should be a single pixel line all across the horizontal width of the window
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: