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)

x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: elig, Assigned: rickg)

References

()

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 ago26 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.