Closed Bug 1699 Opened 26 years ago Closed 26 years ago

<P> tag after </TR> closes table before </TABLE>

Categories

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

All
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: phillip, Assigned: rickg)

References

()

Details

winnt 4.0, win 95 with ngt.exe, build 98113011 the letter from steve case should be in a small table, but it appears at the bottom of the screen. the navigation bar on the left column is also thrown off. this is a highly visible page, especially to our audience: www.mozilla.org...
Summary: ss:terrible layout of the latest news from mozilla.org → terrible layout of the latest news from mozilla.org
Yes, not the prettiest, but acceptable on win95 Nov 30 build. I can read the letter ok. Not ss: for this release. Taking off ss: list for now.
Assignee: buster → rickg
Component: HTMLTables → Parser
see the handy minimized test case below. Note the <P> tags after </TR> in the nested table. These cause closure on the table when they shouldn't. =================== TEST CASE ====================== <HTML> <BODY BGCOLOR="#FFFFFF" TEXT="#000000" LINK="#0000EE" VLINK="#551A8B" ALINK="#FF0000" MARGINHEIGHT=0 MARGINWIDTH=0> <TABLE BORDER=0 CELLPADDING=0 CELLSPACING=0 WIDTH="100%" bgcolor=yellow> <TR> <TD VALIGN=TOP ROWSPAN=2> <P><CENTER> <TABLE WIDTH="80%" BORDER CELLPADDING=16 CELLSPACING=0> <TR> <TD BGCOLOR="#FEFFE6"> <TABLE CELLPADDING=0 CELLSPACING=0 BORDER=0> <TR> <TH VALIGN=BASELINE ALIGN=RIGHT>From: </TH> <TD>Steve Case &lt;...&gt;</TD> </TR> <P> <TR> <TH VALIGN=BASELINE ALIGN=RIGHT>To: </TH> <TD>Jamie Zawinski &lt;<A HREF="mailto:jwz@mozilla.org">jwz@mozilla.org</A>&gt;</TD> </TR> <P> <TR> <TH VALIGN=BASELINE ALIGN=RIGHT>Date: </TH> <TD>Wed, 25 Nov 1998 12:10:18 EST</TD> </TR> <P> <TR> <TH VALIGN=BASELINE ALIGN=RIGHT>Subject: </TH> <TD>Mozilla</TD> </TR> <P> </TABLE> <P>Jamie, </TD> </TR> </TABLE> </TD> </TR> </TABLE> </BODY></HTML>
Status: NEW → ASSIGNED
Duplicate "bad content in table" bug.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → DUPLICATE
Status: RESOLVED → VERIFIED
*** This bug has been marked as a duplicate of 1474 ***
*** Bug 2079 has been marked as a duplicate of this bug. ***
I marked #2079 as a dup of #1699 and here are my comments from there. NOTE: bug #1699 is *NOT* a duplicate of #1474 -- please read the section in #1474 from phillip@netscape.com (11/30/98 11:45) and try the test case there -- it's not the same thing as this bug. - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - This is a dup of #1699 -- aka 'bad content in table' -- there are <P>s in between the <TR>s, and they close out the table wrongly. I have posted a "fixed" version of stevecase.html at http://qsilver.queensu.ca/~buslib/stevecase with a 'diff -c' of how to workaround the problem. However, (I note), in the comments of #1699, bug #1699 has been marked as a duplicate of #1474: This is *not* correct -- look at the test case earlier in the bug report of #1474 -- it is an instance of the <table width="nn%"> bug -- (n.b. there are a lot of duplicates of this bug in the db) The 'bad content in table' and 'table width="nn%"' are *distinct* (although, by their nature, they often travel together in pages with deeply nested tables, and can be confused with each other). The 'dup' of 1699-1474 should be backed-out of the database. Cheers .. John
Status: VERIFIED → REOPENED
Summary: terrible layout of the latest news from mozilla.org → <P> tag after </TR> closes table before </TABLE>
Reopening bug since it is not a dup of 1474. Changing summary to reflect nature of bug more clearly.
Resolution: DUPLICATE → ---
*** Bug 2113 has been marked as a duplicate of this bug. ***
Status: REOPENED → RESOLVED
Closed: 26 years ago26 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Reopening bug since not a dup, but still legitimate. phillip, could you check this with latest build please.
as of the build on friday, 08.jan.1999, it's still as broken as ever on windows. i'm in the process of checking with linux...
same story for RedHat 5.2 linux...
phillip, if this is on Mac too, please change Platform to All.
Hardware: PC → All
verily this happens on my favourite imac too. (at least we have platform parity on this one...). changing to Platform::All
Check this on a post Jan 8th build. I believe that the changes checked in Friday night by rickg@netscape.com were explicitly to address this kind of problem. This page is OK for the 'nightly Jan 12th win95 build'. (Or to be more exact, it is OK *if* you ignore the redundant horizontal scrollbar but that is already covered by several other bug reports and this particular report was direct at the 'bad content in tables' type of bug).
phillip, what build were you on? Was it the latest? Try the 01/11 or 01/12 build and let us know the results against each platform. Thanks!
Sorry if my last 3 comments were unclear. they all applied to the 08.jan.1999 build. i tried it again on NT, MacOS, and Linux, and the table is correctly laid (lain?) out. Linux is harder to verify because it only loads the first page you try. (and then only 640x350 pixels. i'm looking into that.) So this is officially fixed, as far as i'm concerned. (but i'm afraid to mark it as such...)
Status: REOPENED → RESOLVED
Closed: 26 years ago26 years ago
Resolution: --- → FIXED
phillip, I am going to mark this as Resolved/Fixed, you to please mark as verfied. If you have a slightly different new bug with Linux, please write a new bug. If you see this original bug occur in future bugs, please write a new bug.
Status: RESOLVED → VERIFIED
verified as of build 99011312 on NT, RedHat 5.2, and MacOS 8.5
You need to log in before you can comment on or make changes to this bug.