98549 - list substructure is not preserved

Status: VERIFIED DUPLICATE of bug 128793

(Core :: DOM: Editor, defect, P2)

Description

[sujay]

using 7/5 build of netscape

1) launch netscape
2) launch composer
3) make a list like this:

o A
o B
o C
o D
        o 1
        o 2

4) select C thru 2
5) Cut/Copy
6) make a new bullet after "2"
7) Paste

The resulting bullets after the paste are all "flat".
List substructure is not preserved.



------- Additional Comments From [e-mail] 2001-07-05 09:44 -------

resulting list is:

o c
o D
o 1
o 2

I was expecting:

o C
o D
        o 1
        o 2



------- Additional Comments From [e-mail] 2001-07-05 11:07 -------

this should really get into the branch if we can

manager reviewed the need for the fix and agrees, approval for checkin to the
trunk and branch after fix has received an r= and sr=, adding nsBranch keyword



------- Additional Comments From Akkana 2001-07-05 17:05 -------

Here's what I have so far:
in nsHTMLEditor::CreateListOfNodesToPaste(), we create a nsDOMSubtreeIterator
and iterate over it, flat-appending each node to a list from which each node is
then inserted one by one.
That's where the structure is being lost -- the <li><li><ul><li><li> flattens
out to <li><li><li><li> and the intervening <ul> is lost.



------- Additional Comments From [e-mail] 2001-07-09 16:54 -------

putting this back into joe's bug list and setting to 1.0, fix is just to risky 
at this point



------- Additional Comments From Akkana 2001-07-10 14:29 -------

Kin said he thought there might be an easy fix to this, related to some other
recent fixes he's made.  Kin, here's the bug, but feel free to send it back if
you don't have time.

Comment 1

[sujay]

*** Bug 89403 has been marked as a duplicate of this bug. ***

Comment 2

[kinmoz]

--> pulling this into mozilla0.9.8

Comment 3

[Syd Logan]

list problem, plussing

Comment 4

[kinmoz]

--> mozilla1.0

Comment 5

[Kevin McCluskey (gone)]

This looks like a dup of http://bugzilla.mozilla.org/show_bug.cgi?id=98547

Comment 6

[kinmoz]

yeah it's probably a dup, it only happens if the beginning and end of the
selection are at 2 different levels ... I remember fixing or a reviewing a fix
like this a while ago, but I think it has to do with alignment or indenting.

*** This bug has been marked as a duplicate of 98547 ***

Comment 7

[sujay]

v

Comment 8

[Joe Francis]

It's not a dup.  Well, ok, it *is* a dup.  But of a different bug.  128793.  
98547 is about changing list types.  This bug (and 128793) are about pasting.

Ends up being a different code path.  re-duping to correct bug.

Comment 9

[Joe Francis]

*** This bug has been marked as a duplicate of 128793 ***

Comment 10

[sujay]

v