Closed Bug 50099 Opened 24 years ago Closed 24 years ago

dd is flow, not inline

Categories

(Core :: DOM: HTML Parser, defect, P3)

defect

Tracking

()

RESOLVED DUPLICATE of bug 57634

People

(Reporter: BenB, Assigned: rickg)

Details

(Whiteboard: [nsbeta3 +] fix waiting)

Reproduce:
1. Create a page with the following content and open it in the editor or write
the following in the HTML source pane in the editor:

<dl><dt>Security
</dt><dd>

<dl><dt>OpenPGP
</dt><dd>Key DE08AD92
</dd><dt>S/MIME
</dt><dd>Certificate available on request
</dd></dl>

</dd>
</dl>

(i.e. a <dl> nested in a <dd>)

2. Open the preview or normal pane, so the source gets parsed.
3. Go (back) to the HTML source pane

Actual result:
You see

<dl><dt>Security
</dt><dd>
</dd><dl><dt>OpenPGP
</dt><dd>Key DE08AD92
</dd><dt>S/MIME
</dt><dd>Certificate available on request
</dd></dl></dl>

(i.e. the inner <dl> has been move outside the <dd>)

Expected result:

You see the same source you entered in step 1.

Relevance:
Gecko renders both the same, but the HTML->TXT converter will (correctly) render
it, as if the inner <dl> were a sibling of the outer <dl>. This is a change in
the meaning, i.e. data loss.
Severity: normal → major
Keywords: nsbeta3
yes, this is clearly incorrect, the nested DL should not be moved. Assigning to 
jfrancis

Actually, in normal mode, I can't nest a definition list within a dd, when I 
select to insert a dt in a dd it pushes it out as a sibling.
Assignee: beppe → jfrancis
Whiteboard: [nsbeta3+]
Target Milestone: --- → M18
parser issue, folks.
Assignee: jfrancis → rickg
Component: Editor → Parser
QA Contact: sujay → janc
*** Bug 48376 has been marked as a duplicate of this bug. ***
Fixed in my tree.
Status: NEW → ASSIGNED
Whiteboard: [nsbeta3+] → [nsbeta3+] fix waiting
Corrected the flaw in the elementtable.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
This has regressed and is now covered by bug 57634.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

*** This bug has been marked as a duplicate of 57634 ***
Status: REOPENED → RESOLVED
Closed: 24 years ago24 years ago
Resolution: --- → DUPLICATE
Whiteboard: [nsbeta3+] fix waiting → [nsbeta3 +] fix waiting
updated qa contact.
QA Contact: janc → bsharma
QA Contact: bsharma → moied
You need to log in before you can comment on or make changes to this bug.