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)
Tracking
(Not tracked)
VERIFIED
FIXED
M4
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.
Reporter | ||
Comment 1•26 years ago
|
||
[As a friendly FYI, this is still taking place using 12.4.98a build, using
12.7.98 news page from ~9:35 AM.]
Reporter | ||
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → WONTFIX
Reporter | ||
Updated•26 years ago
|
Status: RESOLVED → REOPENED
Reporter | ||
Comment 2•26 years ago
|
||
As a friendly FYI, still taking place on 12.30.98 build.
Reporter | ||
Updated•26 years ago
|
Resolution: WONTFIX → ---
Reporter | ||
Updated•26 years ago
|
Status: REOPENED → ASSIGNED
Reporter | ||
Comment 3•26 years ago
|
||
[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.]
Comment 4•26 years ago
|
||
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>");
----------------------------------------------------------------------
Reporter | ||
Comment 5•26 years ago
|
||
Wow; great sleuthing, John!!!
Updated•26 years ago
|
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)
Comment 7•26 years ago
|
||
Setting resolution to Fixed, since cited URL and test cases seem to work in
latest build.
Reporter | ||
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 8•26 years ago
|
||
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.)
Updated•16 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•