6247 - incorrect table layout

Status: VERIFIED FIXED

(Core :: Layout, defect, P3)

Description

[edburns]

The TH Browser doesn't have anything beneath it.  It should have some TD's beneath it.  Try loading this page in NS4.x and mozilla (buildid 1999050814):

http://java.sun.com/people/edburns/mozbug.html

Comment 1

[John Morrison]

Hmm. You have closed out the first row with a </TR> right after
'Engineering .. Frameworks .. Phone' in the 'Apps Groups' TH/Column.
So, Mozilla 5.0 seems to do things correctly.

What I can't see is why Nav4.5 _doesn't_ close out the same row.

  <TABLE BORDER>
    <TR>
      <td> (1,1) </td>
      <td> (1,2) </td>
      <td> (1,3) </TR>     <!-- why does 4.5 ignore this TR -->
      <td> (1,4) </td>
    </TR>
  </TABLE>

Comment 2

[John Morrison]

Attachment: testcase; dubious [PP] -- ending table rows with </TR>

Comment 3

[rickg]

Egads. The </TR> is being ignored by 4.x. This is strictly incorrect, but I may
have to do it for backward compatibility.

Comment 4

[John Morrison]

BTW, IE 5.0 has the same behaviour (displays as a single row).

Which leads me to ask ... if Nav 4.5 and IE interpret this as a 1x4 table,
is it *me* who is out to lunch here (on the face of it, I think not, but it
seems like such a fundamental thing).

Comment 5

[rickg]

Corrected by changes to parsing engine.

Comment 6

[John Morrison]

*** Bug 6990 has been marked as a duplicate of this bug. ***

Comment 7

[John Morrison]

Bug #6990 pointed out that TH wants the same treatment as TD ;)
Tested with May23 viewer win95

  <table border>
  <tr>
    <th> foo </TR> </th>  <!-- Nav ignores this  /TR -->
    <th> bar </th>
    <th> bar
    <th> bar </th>
  </tr>
  </table>

Comment 8

[Christine Hoffman]

Testing with 5/24 Apprunner, bug is fixed for <TD> and should be fixed for <TH>
with tomorrow's build. Will check tomorrow to verify.

Comment 9

[Christine Hoffman]

Bug not fixed for <TH> in 5/25 build. Will try tomorrow.

Comment 10

[Christine Hoffman]

Testing 6/3 with Apprunner, the bug is resolved for <TD> but not for
<TH>. Reopening bug.

Comment 11

[rickg]

Sorry chris-- try it again -- the fix went in last night.

Comment 12

[Christine Hoffman]

Using 6/17 build, bug resolved for <td> and <th>. Verifying bug fixed.