Closed
Bug 22483
Opened 25 years ago
Closed 25 years ago
bad display (html codes appearing)
Categories
(Core :: DOM: HTML Parser, defect, P3)
Tracking
()
People
(Reporter: chagam, Assigned: rickg)
References
()
Details
(Keywords: verifyme)
Attachments
(1 file)
|
1.03 KB,
text/html
|
Details |
Hi.
I already had problems whith page with netscape Navigator (even 4.7)
1)it seems that the add banner is the source of problem N°1. this banner doesnt
appear on the screen but we can see html code instead. the probleme is
reproductible (just go to http://www.3dfxmania.com to see it)
2)top of all sections (Downloads / The info / The rest...) is badly displayed
corners should be rounded correctly.
Chag
|
||
Updated•25 years ago
|
Assignee: nobody → rickg
Component: Browser-General → Parser
Summary: bad display (html codes appearing) and bad images displaying → bad display (html codes appearing)
|
|
||
Comment 1•25 years ago
|
||
Basically, this breaks parsing:
<SCRIPT LANGUAGE="JavaScript">
document.write('<SCRIPT SRC="http://www.foo.com/'+args+'"></SCRIPT>');
</SCRIPT>
--------------------------------------------------------------------^
I'm attaching the full script to this bug for your testing. I'll file the
other problem (tables) separately.
|
|
||
Comment 2•25 years ago
|
||
|
|
||
Comment 3•25 years ago
|
||
The other bug is bug #22513 (HTMLTables, karnaze).
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → INVALID
We treat this incorrect HTML the same as navigator. If you plan to include
the </SCRIPT> tag in script, you must escape it. Please refer to your javascript
documentation.
|
|
||
Comment 5•25 years ago
|
||
Every time I think there some bit of horrendous HTML that maybe shouldn't be
fixed, you go off and fix it. Now you cross me up!
It is certainly invalid HTML, but (I believe) this example shows that Nav4.7
handles it without breaking. (Your option, of course, on what to do :-])
<html><body>
Here is var bar<br>
<SCRIPT LANGUAGE="JavaScript">
<!--
var foo = 'var bar = "BAZ"';
document.write('<SCRIPT>' + foo + '</SCRIPT>');
document.write('"var bar" has value |' + bar + '| <br>\n');
// -->
</SCRIPT>
That was var bar<br>
</body></html>
The original testcase was invalid, but the new one is legit (since it's in
comments). Reopening. Sigh.
Rick, SCRIPT contents enclosed within a "comment" does not mean anything to the
parser. As far as the parser is concerned contents within SCRIPT should be
treated as CDATA. Therefore, I would suggest to invalidate this bug!!
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → INVALID
Too much eggnog, I think. Harish is right, and this is also invalid HTML. See my
earlier remarks.
|
|
||
Comment 9•25 years ago
|
||
rickg, me and harishd (hey, it rhymes) agree that unescaped /SCRIPT inside
SCRIPT is undeniably invalid HTML, and, yes, comments are irrelevant within
CDATA. It's also, I'm sure, a very painful pattern to parse.
However, Nav4.6/4.7 on win32 "forgives" this abuse (and it's probably XP for
4.0n+, at least).
So, at risk of your goodwill, I am going to REOPEN this bug just to bring the
the 4.xP point to your attention (but immediately INVALID this bug again if
I need a whack with the clue stick -- that is quite OK with me [I'm not the
original bug reporter anyways]).
This is only a backwards-compatibility issue. Unfortunately, most people code
to what works, and, really unfortunately, it's ad banners that may be the
biggest abusers of this "forgiveness": i.e., I think this will be reported "as
a bug" quite a few times as you move into beta.
|
|
||
Updated•25 years ago
|
Status: RESOLVED → REOPENED
|
|
||
Comment 10•25 years ago
|
||
Doh! Didn't hit the REOPEN radio last time ...
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: INVALID → REMIND
|
Assignee | |
Comment 11•25 years ago
|
||
Ok -- I'll leave it in the remind pile, only because you've been a big help to
us. I'll think about it later under the category "ways to improve dealing with
crappy html.". : )
|
||
Comment 12•25 years ago
|
||
There is another bug covering this issue now...
Status: RESOLVED → REOPENED
Resolution: REMIND → ---
|
|
||
Comment 13•25 years ago
|
||
...and this is a duplicate.
*** This bug has been marked as a duplicate of 26857 ***
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Keywords: verifyme
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•