Closed
Bug 1671
Opened 26 years ago
Closed 26 years ago
Table layout incorrect
Categories
(Core :: DOM: HTML Parser, defect, P2)
Tracking
()
VERIFIED
FIXED
People
(Reporter: michael.j.lowe, Assigned: rickg)
References
()
Details
Table cells on this page are shifted down and to the left of the page starting
from the "links.gif" gif image (as compared to other web browsers).
This is a parser bug. The table is getting closed prematurely. If you remove
the first font tag from the test case below, it works correctly.
=========== TEST CASE ================
<HTML><BODY BGCOLOR="white" VLINK="#336666" LINK="#0033cc" ALINK="purple">
<center>
<table border="0" CELLPADDING="0" CELLSPACING="0" WIDTH="611" bgcolor=aqua>
<tr>
<td>
<dl>
<font face="geneva,arial" size="2">
<dt><a href="about/about.htm">About the
University of Newcastle</a><br>
</font>
<dd>
<table border="0" bgcolor=yellow>
<tr>
<td><font face="geneva,arial"
size="1">Central Coast Campus</font></td>
</tr>
<tr>
<td><font face="geneva,arial"
size="1">Facts and figures, Staff Directory.</font></td>
</tr>
<tr>
<td><font face="geneva,arial"
size="1">Principal Officers, Faculties and Departments.</font></td>
</tr>
</table>
</dd>
</dt>
</dl>
</td>
</tr>
</table>
stuff after table
</CENTER>
</BODY>
</HTML>
|
Reporter | |
Comment 2•26 years ago
|
||
Here is a patch which seems to fix this bug along with a seperate parser bug
which occurs with the page http://www.cs.newcastle.edu.au because one of their
<A HREF=> </A> tags is not closed properly.
Index: CNavDTD.cpp
===================================================================
RCS file: /cvsroot/mozilla/htmlparser/src/CNavDTD.cpp,v
retrieving revision 3.119
diff -u -r3.119 CNavDTD.cpp
--- CNavDTD.cpp 1998/12/08 17:33:28 3.119
+++ CNavDTD.cpp 1998/12/09 16:40:57
@@ -1402,7 +1402,7 @@
case eHTMLTag_dl:
{
static eHTMLTags
okTags[]={eHTMLTag_dd,eHTMLTag_dt,eHTMLTag_whitespace,
-
eHTMLTag_map,eHTMLTag_form,eHTMLTag_newline,eHTMLTag_p};
+
eHTMLTag_map,eHTMLTag_form,eHTMLTag_newline,eHTMLTag_p,eHTMLTag_font};
result=FindTagInSet(aChild,okTags,sizeof(okTags)/sizeof(eHTMLTag_unknown));
}
break;
@@ -1811,7 +1811,7 @@
* @return PR_TRUE if autoclosure should occur
*/
PRBool CNavDTD::RequiresAutomaticClosure(eHTMLTags aParentTag,eHTMLTags
aChildTag) const {
- static eHTMLTags
gAutoCloseTags[]={eHTMLTag_li,eHTMLTag_td,eHTMLTag_tr,eHTMLTag_dt};
+ static eHTMLTags
gAutoCloseTags[]={eHTMLTag_li,eHTMLTag_td,eHTMLTag_tr,eHTMLTag_dt,eHTMLTag_a};
PRBool result=PR_FALSE;
PRInt32 theParentIndex=kNotFound;
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
This bug is fixed, but this page also shows the "wrong font" bug (a dup).
|
|
Reporter | |
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•