Closed
Bug 69098
Opened 24 years ago
Closed 21 years ago
layout error when inserting <hr><br><hr> via DOM
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
WORKSFORME
Future
People
(Reporter: Brade, Unassigned)
Details
(Keywords: css1, dom1, testcase, Whiteboard: [Hixie-P4])
Attachments
(2 files)
In Composer, I came across this case where inserting an <hr>, pressing return
(which generates a <br> in Composer today), and then inserting another <hr>
didn't render the same as it did in the browser. After much further
investigation and experimentation, I am filing this bug. I have two attachments
to add which can be used to experiment without using Composer. Those will follow.
Note: In the one testcase, when adding the tags to the browser via
document.write, the browser renders correctly (a bigger gap between the two <hr>
tags). In the other testcase, when adding the tags via the dom, the browser
shows the two <hr> close together (ignoring a <br> in between).
Marc--I'd be happy to do some preliminary legwork on this bug but I need a few
pointers.
Reporter | ||
Comment 1•24 years ago
|
||
Reporter | ||
Comment 2•24 years ago
|
||
Comment 3•24 years ago
|
||
I'm building right now, but the first thing I need to check is how the content
model looks via 'Dump Content' in Viewer. Maybe you have already done this, but
we need to know if the BR is getting inserted into the content model before I
can get anywhere with the potential layout problems. If it is, then I'll check
what frames are getting created, and if not, then it is likely a DOM or
ContentSink problem.
BTW: I noticed that this is broken in NS6 RTM as well. How critical is this to
composer? Should we nominate for Moz 0.9?
... Accepting.
Status: NEW → ASSIGNED
Summary: layout error when inserting <hr><br><hr> via DOM → layout error when inserting <hr><br><hr> via DOM
Comment 4•24 years ago
|
||
brade says it is pretty important - marking P2 and Mozilla 0.9
Priority: -- → P2
Target Milestone: --- → mozilla0.9
Comment 5•24 years ago
|
||
The problem is with the BR being appended after the HR in Quirks mode. I think
it is a problem because the HR in Quirks mode is really an Inline with a \A
after it (using HR:after { content: "\A"; }) and when the BR is appended after
the line has already been reflowed, it is not correctly reevaluating the lines
and reflowing the BR down to its own line. In STD mode it is OK because HR is a
block. My guess is that other cases where content:"\A" is used will probably
show the same problem.
I'm debugging this now, but it is all new code to me, so patience or assistance
is appreciated...
Updated•24 years ago
|
Component: Layout → DOM Core
QA Contact: petersen → desale
Whiteboard: [Hixie-P4]
Comment 6•24 years ago
|
||
Downgrading slightly - moving to Mozilla 1.0 to make room for top crashers in
Mozilla 0.9.1
Priority: P2 → P3
Target Milestone: mozilla0.9 → mozilla1.0
Updated•23 years ago
|
QA Contact: desale → stummala
Comment 7•23 years ago
|
||
Updating QA contact to Shivakiran Tummala.
Comment 9•21 years ago
|
||
I know it's gonna be done soon automatically but I'm sick of the target
milestone and assignee :-)
Besides I don't understand the testcases
Assignee: attinasi → general
Severity: major → normal
Status: ASSIGNED → NEW
Priority: P3 → --
Target Milestone: mozilla1.1alpha → Future
Comment 10•21 years ago
|
||
<hr> got whacked and the testcases both work identically for me... is this still
a problem, brade?
Reporter | ||
Comment 11•21 years ago
|
||
I don't know if this is still an issue or not. I wouldn't be surprised if it
was fixed with some other <hr> bug. I think that this can be resolved as
worksforme.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → WORKSFORME
Component: DOM: Core → DOM: Core & HTML
QA Contact: stummala → general
You need to log in
before you can comment on or make changes to this bug.
Description
•