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)

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

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: phillip, Assigned: rickg)

References

(
URL
)

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 ago → 26 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 ago → 26 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.