Closed Bug 1754 Opened 26 years ago Closed 26 years ago

"News" only presents left navigation column (from preview toolbar button)

Categories

(Core Graveyard :: Viewer App, defect, P2)

Product:
Core Graveyard
Component:
Viewer App
Platform:
x86
Windows NT
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: elig, Assigned: rickg)

References

(
URL
)

Details

Pressing News (12.1.98 build on Win NT 4 SP3) yields a page containing a banner
ad, left-column navigation bar, and two grey rectangles (about 8x16 pixels)
towards the top of the page.

A number of JavaScript syntax errors are listed in the stdout-like window.
Status: NEW → ASSIGNED
[As a friendly FYI, this is still taking place using 12.4.98a build, using
12.7.98 news page from ~9:35 AM.]
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → WONTFIX
Status: RESOLVED → REOPENED
As a friendly FYI, still taking place on 12.30.98 build.
Resolution: WONTFIX → ---
Status: REOPENED → ASSIGNED
[Urgh. Accidentally WONTFIX'd this bug while touching another bug, due to
Bugzilla's suboptimal UI. Re-opened/assigned, which I believe to be its prior
state.]
The following tage pattern closes out tables prematurely, and only the
first cell of the table is rendered. I think this is the layout
problem noted for Bugzilla #1754 as well as #1095 & #1266. (This
pattern is used on most of Netscape's "channel" pages (e.g. news,
download, local, family, etc.)).

 i.e.,  <TABLE><TR><TD>
              This TD will be displayed.
              <IFRAME SRC>
                    <SCRIPT SRC></SCRIPT>
              </IFRAME>
          </TD><TD>
              This TD will NOT be displayed (it is discarded).
         </TD></TR></TABLE>

NOTE: it is specifically having a SCRIPT inside the IFRAME that breaks
the table; other tags do not break the table.

Here's the test case:
  ----------------------------------------------------------------------
<html><head></head><body>   <br><br>

<TABLE border WIDTH=600 cellpadding=10 align=center>
<TR>

<TD>
<IFRAME WIDTH=142 HEIGHT=72
        SRC="http://qsilver.queensu.ca/~buslib/ns-news/iframe-src.html">
  <!-- for HTML 4.0/NGLayout, this payload should be ignored -->
  What's an IFRAME?<br>
  <!-- if you either :
         1) remove the whole <SCRIPT></SCRIPT> phrase below, or
         2) delete the SRC attribute from this SCRIPT tag, or
         3) simply change <SCRIPT> to <fooSCRIPT> (i.e., unknown tag),
       then the next TD in this table is correctly rendered. It appears
       that this SCRIPT is prematurely closing out the table.
    -->
  <script language=javascript
        src="http://qsilver.queensu.ca/~buslib/ns-news/addyn.js"></script>
</IFRAME>
</TD>

<TD>
<IFRAME WIDTH=142 HEIGHT=72
        SRC="http://qsilver.queensu.ca/~buslib/ns-news/iframe-src.html">
  What's an IFRAME?<br>
  <script language=javascript
        src="http://qsilver.queensu.ca/~buslib/ns-news/addyn.js"></script>
</IFRAME>
</TD>

</TR>
</TABLE>
</body></html>

  ----------------------------------------------------------------------
You can look at a simple test case at :
  http://qsilver.queensu.ca/~buslib/ns-news/ns-news.broken-table.html
and some other examples at:
  http://qsilver.queensu.ca/~buslib/ns-news/

NOTE:

 http://qsilver.queensu.ca/~buslib/ns-news/iframe-src.html contains:
  <IMG SRC="http://www.mozilla.org/newlayout/testcases/mozilla_bungee.gif"
       WIDTH=71 HEIGHT=144 border=1>

 http://qsilver.queensu.ca/~buslib/ns-news/addyn.js contains:
  document.write("<IMG SRC=\"http://www.mozilla.org/newlayout/l.gif\" WIDTH=117
HEIGHT=120 border=1>");

  ----------------------------------------------------------------------
Wow; great sleuthing, John!!!
Setting all current Open/Normal to M4.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago → 26 years ago
Resolution: --- → FIXED
Summary: "News" only presents left navigation column (from preview toolbar button) → "News" only presents left navigation column (from preview toolbar button)
Setting resolution to Fixed, since cited URL and test cases seem to work in
latest build.
Status: RESOLVED → VERIFIED
Yup. I believe that Michael is correct.

Verified fixed both on the news page, and using John's most excellent test case.
(Performed verifications on 2.8.99 Mac OS build, as well as 2.4.99 Win32 and
Linus builds.)
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.