59185 - Page Renders Incorrectly--side bar is way to wide

Status: VERIFIED FIXED

(Tech Evangelism Graveyard :: English US, defect, P3)

Description

[Igor]

1. Open the page

Error-the side bar on this page doesn't seem to have limit on how wide it can
be.  I don't know if this is a problem with the site or with the browser,
however it does render correctly in Netscape 4.75.  It should render as three
columns.  With the blue left side bar taking up only about 1/4 of the width of
the screen.

RedHat 7.0, Kernel 2.2.16-22 Build 2000110421 X-Free86 4.0

Comment 1

[Boris Zbarsky [:bzbarsky]]

The HTML on the page is malformed as a result of the following JavaScript:

<script language="javascript">
if(document.layers){
   document.write("<table width='100%' border='0' cellspacing='0'
cellpadding='0' bgcolor='#cccc99'>");
}
if(document.all){
   document.write("<table width='100%' border='0' cellspacing='0'
cellpadding='0' background='/images/bg_tantop.gif'>");
}
</script>

Since Mozilla supports neither the non-standard document.layers construct nor
the non-standard document.all construct, no <table> tag is written at all.

If the above javascript is just replaced with a <table> tag, the page renders
correctly.

The page also contains other instances of JavaScript that makes the erroneous
assumption that all Netscape browsers support document.layers, but these do not
result in invalid HTML...

The site should change its JavaScript to be compliant with the W3C DOM.  Or just
put an "else" clause in those "if" statements.

I suggest reassigning this to Evangelism.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Mail sent to site administrator describing the problem and suggesting some
solutions, as well as pointing them at
http://sites.netscape.net/ekrock/standards.html

Comment 3

[timeless]

cool, so someone from evang will mark fixed when the site admin corrects the 
code.

Comment 4

[Blake Ross]

-> [e-mail] for my evangelism bugs.

removing the now-depreciated evangelism-related keywords.

setting platform to All.

Comment 5

[Igor]

The one of developers at the website wrote me indicating that they had fixed the
problem.  Closing as Fixed--If that's the wrong way to close out an evangelism
bug please correct me.

Comment 6

[Boris Zbarsky [:bzbarsky]]

Looks fixed to me.  In the code, they have added an "else" branch to the
layers/all test.

verifying fixed.

Comment 7

[Bob Clary [:bc] (inactive)]

All Evangelism Bugs are now in the Product Tech Evangelism. See bug 86997 for
details.