Closed
Bug 40855
Opened 25 years ago
Closed 24 years ago
comment interpreted as <br> when inside <table><center>
Categories
(Core :: DOM: HTML Parser, defect, P3)
Core
DOM: HTML Parser
Tracking
()
VERIFIED
FIXED
mozilla0.9
People
(Reporter: xalkina, Assigned: harishd)
References
()
Details
(Keywords: compat, relnote, testcase, Whiteboard: nsbeta3 relnote-devel)
Attachments
(5 files)
at bottom of page, there's a "page was updated <date>" entry. in the source
there's a comment between the text and the date. mozilla renders as if there was
a <br>.
linux-2000052520
Reporter | ||
Comment 1•25 years ago
|
||
Comment 2•25 years ago
|
||
Comment 3•25 years ago
|
||
Comment 4•25 years ago
|
||
Christos' testcase is simply the full page...thats what the URL field is for
Christos.
Problem appears to be with text inside a <center> block thats inside a <table>,
which doesn't seem like it would be valid HTML. Sun's pages are absolutly
DISGUSTING, it's filled with this kind of cruft.
Attaching minimal test case.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: testcase
Summary: comment interpreted as <br>?? → comment interpreted as <br> when inside <table><center>
Comment 5•25 years ago
|
||
Comment 6•25 years ago
|
||
Arg, collitions... gotta start using makingtest. My testcase is more minimal,
anyhow.
Comment 7•25 years ago
|
||
Yes, I got a collision, too. Will start using `makingtest', too.
The only thing left from my collision-comment is that 4.x displays
all testcases as expected on one line.
Comment 8•25 years ago
|
||
Comment 9•25 years ago
|
||
The above testcase makes this look much more like a Parser problem (although I
don't have access to viewer now, so I can't tell for sure). Changing to Parser.
Assignee: clayton → rickg
Component: Layout → Parser
QA Contact: petersen → janc
Assignee | ||
Comment 11•25 years ago
|
||
The layout does look bad...but..the bug as such is not a high priority. In other
words it's not a beta blocker. Moving to M18.
Status: NEW → ASSIGNED
Target Milestone: --- → M18
Reporter | ||
Comment 12•25 years ago
|
||
Works fine in linux/2000052920 in all pages at javasoft I checked
Reporter | ||
Comment 13•24 years ago
|
||
see it again as of 2000060908 on linux, there are some pages @ Sun however that
now do it with <p align="center"> instead of <center> and there it works fine.
Comment 14•24 years ago
|
||
Egads. This is due to SaveTokens(). The <center>, text, and comment are inside a
<table> (malformed). They're supposed to get pushed out, but for some reason the
<center> is closed when the comment appears. Ick, ick, ick. Bad Harish.
Assignee | ||
Comment 15•24 years ago
|
||
That's because comments, any where insided a table, are treated as legal. And
why shouldn't it be?????? Anyway, the fix would be trivial.
I think you would see the same effect if there was a CAPTION inside
CENTER instead of the comment.
Assignee | ||
Comment 16•24 years ago
|
||
Recommended for nsbeta3. Comments should not be the culprit in handling illegal
table contents.
Assignee | ||
Comment 17•24 years ago
|
||
This bug has been marked "future" because the original netscape engineer working
on this is over-burdened. If you feel this is an error, that you or another
known resource will be working on this bug,or if it blocks your work in some way
-- please attach your concern to the bug for reconsideration.
Target Milestone: M18 → Future
Comment 18•24 years ago
|
||
Marking nsbeta3-
Updated•24 years ago
|
Whiteboard: [nsbeta3-]
Comment 19•24 years ago
|
||
I think we should reconsider this. When bogus content is found in a <table>
before <tr>, it gets pushed out like nav did it. For some reason, text and
comments aren't getting moved. This could lead to pretty bad (and
non-compatible) behavior.
Whiteboard: [nsbeta3-] → nsbeta3
Assignee | ||
Comment 20•24 years ago
|
||
*** Bug 52012 has been marked as a duplicate of this bug. ***
Comment 21•24 years ago
|
||
All/all. I can reproduce this on Windows 98, 2000093008.
Should this get a release note?
Updated•24 years ago
|
Whiteboard: nsbeta3 → nsbeta3 relnote-devel
Comment 22•24 years ago
|
||
Nom. nsbeta1. We should handle HTML comments correctly. ;->
Keywords: nsbeta1
Assignee | ||
Comment 23•24 years ago
|
||
The bug seems to have gone away! Jan, could you please verify this. Thanx.
Marking WFM.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → WORKSFORME
Comment 24•24 years ago
|
||
Still broken on MTEST builds:
2001-01-31-14-MTEST : Win2K
2001-01-30-21-MTEST : Mac
but working fine on trunk builds:
2001-02-02-10-Mtrunk : Win98
2001-02-02-10-Mtrunk : Linux
2001-02-02-09-Mtrunk : Mac
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Reporter | ||
Comment 26•24 years ago
|
||
wfm2, on both linux/2001020306 & win/2001020308
Assignee | ||
Comment 27•24 years ago
|
||
Fix is in. Marking FIXED.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
Comment 29•23 years ago
|
||
Verified on
build: 2001-06-20-04-Trunk
platform: Win NT
The above test case and the test case with id 9299 loads fine.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•