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.