Online Gaming Review homepage does not layout properly

VERIFIED WORKSFORME

Status

()

Core
Layout
P2
normal
VERIFIED WORKSFORME
20 years ago
18 years ago

People

(Reporter: tcrowe, Assigned: rickg)

Tracking

Trunk
Future
x86
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

20 years ago
Please take a look at this page.  It does not layout properly (and never
has as far as I know) using nglayout.  It looks like the table columns are not
being spaced out properly.

Updated

20 years ago
Assignee: kipp → buster

Comment 1

20 years ago
This seems to be table layout hell; Alternatively, it's a parser bug. Steve will
know which it is, so I'm assigning it to him.

Updated

20 years ago
Assignee: buster → rickg

Comment 2

20 years ago
This page is full of some truly aweful HTML.
I've reduced the page down to two primary problems.
1. The first is a <br> tag between table rows.  This seems to split the table.
There is a comment in the test case below for Rick to remove the offending <br>
to see the problem.
2. The second problem is much less clear, probably a closure bug in the parser.
There is a comment in the test case below for Rick to remove the <CENTER> tag to
see the effect of this bug.

================  test case  ===================
<html><BODY topmargin="0" leftmargin="0" BGCOLOR="#ffffff" LINK="#7d5111"
ALINK="#0099cc" vlink="#c8811b">

<TABLE WIDTH="604" BORDER="0" CELLSPACING="0" CELLPADDING="0">
<TR>
	<TD ROWSPAN="4" WIDTH="130" VALIGN="TOP" BGCOLOR="yellow">xxx</TD>
	<TD WIDTH="5" ROWSPAN="4" VALIGN="TOP" BGCOLOR="#ffffff">yyy</TD>
	<TD WIDTH="464" ROWSPAN="4" VALIGN="TOP" ALIGN="CENTER"
BGCOLOR="#ffffff">
        <table width="455" border="0" cellspacing="2" cellpadding="2">
          <tr>
            <td width="100%">
				<center><a href="/previews/wargasm_1.phtml"><img
width=450 height=150
SRC="http://www.ogr.com/graphics/homepage/wargasm-homepage.jpg" border="0"
alt="Wargasm Preview"></a><font size="-1" face="verdana,arial"></center></font>
			</td>
          </tr>
<!-- RICK: remove this tag, and problem 1 goes away --><br>
          <tr>
            <td width="100%" bgcolor="#d9dea2"></td>
		  </tr>
		  <tr>
			<td width=100% align=top></td>
		  </tr>

		  <tr>
			<td>
<!-- RICK: remove this tag, and problem 2 goes away --><CENTER>
				<TABLE WIDTH="455" BORDER="0" CELLSPACING="3"
CELLPADDING="0">
					<TR>
						<TD WIDTH="151" VALIGN="TOP"
BGCOLOR="#84ce00" ALIGN="CENTER">
							<a
href="/reviews.html"><IMG SRC="http://www.org.com/graphics/core/reviews-l.gif"
alt="Latest Reviews" WIDTH="94" HEIGHT="29" ALIGN="BOTTOM" border=0></a><BR>
							<FONT SIZE=-1>
							<a
href="/hardware/marvel_g200_1.phtml"><font face=arial color=#000001>Marvel
G200</a></font><br>
							<font face=arial
color=#000001>Intensor Gaming Chair</font><br>
							</FONT>
						</TD>
					</tr>
				</table>
			</td>
		  </tr>
        </table>
    </TD>
	<TD WIDTH="5" ROWSPAN="4" VALIGN="TOP" ALIGN="RIGHT"
BGCOLOR="#ffffff"></TD>
</tr>
</table>
</FORM>
</BODY>
</html>
(Assignee)

Updated

20 years ago
Status: NEW → ASSIGNED

Comment 3

20 years ago
[Just for the record, it's still rendering wackily and differently from Comm
4.x using 11.23a.98 Seamonkey builds.]
(Assignee)

Updated

20 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 20 years ago
Resolution: --- → FIXED

Updated

20 years ago
QA Contact: 1698

Comment 4

20 years ago
Verified fixed on 2.8.99 Mac OS build.

However, it doesn't look right on the 2.4.99 Win32 or Linux builds; holding to
formally mark as verified until the 2.9.99 builds are available.

Updated

20 years ago
Status: RESOLVED → VERIFIED

Comment 5

20 years ago
Looks great on the 2.9.99 Win32 & Linux builds. (okay, there was this small issue
of a Win32 blue screen while scrolling down the page, but that's unrelated. ;)

Thus, verified.

Comment 6

18 years ago
Fix was bad, DOM events will be null because is on the way *in* from frames, 
meaning we havn't build the DOM event yet. My bad (saari typing) for not 
catching this!
Status: VERIFIED → REOPENED
Resolution: FIXED → ---

Comment 7

18 years ago
Doesn't sound like a very high priority bug. Setting to FUTURE.
Status: REOPENED → ASSIGNED
Target Milestone: --- → Future

Comment 8

18 years ago
This lays out fine in Linux 2000-08-13-09.  This bug is old and grey.  Eli
previously WFM'd this on Windows and Mac.  The only thing I am confused by is
saari's comment.  I think that was meant for another bug.  The next person who
dusts this one off should resolve, IMHO.
(Assignee)

Comment 9

18 years ago
This appears to work fine now-- not as the result of any known bug fix.
Status: ASSIGNED → RESOLVED
Last Resolved: 20 years ago18 years ago
Resolution: --- → WORKSFORME

Comment 10

18 years ago
Works for Me
Platform: PC
OS: Windows 98
Build # 2000100508 M18 Trunk Build
Marking as Verified



Ugly ass homepage btw :)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.