Closed Bug 1205 Opened 26 years ago Closed 24 years ago

Online Gaming Review homepage does not layout properly

Categories

(Core :: Layout, defect, P2)

Product:
Core
Component:
Layout
Platform:
x86
Windows NT
Type:
defect

Tracking

()

Status:
VERIFIED WORKSFORME
Milestone:
Future

People

(Reporter: tcrowe, Assigned: rickg)

References

(
URL
)

Details

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.
Assignee: kipp → buster
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.
Assignee: buster → rickg
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>
Status: NEW → ASSIGNED
[Just for the record, it's still rendering wackily and differently from Comm
4.x using 11.23a.98 Seamonkey builds.]
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
QA Contact: 1698
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.
Status: RESOLVED → VERIFIED
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.
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 → ---
Doesn't sound like a very high priority bug. Setting to FUTURE.
Status: REOPENED → ASSIGNED
Target Milestone: --- → Future
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.
URL: www.ogr.com → http://www.ogr.com
This appears to work fine now-- not as the result of any known bug fix.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago → 24 years ago
Resolution: --- → WORKSFORME
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.