87882 - pasting text from one bullet to another yields extra bullet

Status: VERIFIED FIXED

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

Description

[sujay]

Joe, Beth sent you a mail note on this issue. I know you intended this by
design, but....

using 6/25 build of netscape

1) launch netscape
2) Open Kevin McCluskey's attachment in this bug report 57477
3) Bring it into the editor
4) select the <Move under heading1> text
5) ctrl-c to copy to clipboard
6) click on the <LI> under heading 1
7) ctrl-v to paste

it pastes in fine, however you end up with an extra bullet that you
didn't ask for.

all platforms.

Comment 1

[sujay]

adding QAHP

Comment 2

[shrirang khanzode]

is this any different than bug 86766 ? 

Comment 3

[kinmoz]

Added URL for test case sujay mentioned above:

    http://bugzilla.mozilla.org/showattachment.cgi?attach_id=17686

FYI, it's not adding an "extra" bullet, it's simply pushing the existing blank 
list item below your newly pasted list items. I think the assumption is *not* to 
merge the contents of existing list items with the contents of list items being 
pasted in, which in my opinion is a safe assumption since you can't tell what 
the user intended.

I think this is what the current rules are set up to do ...


  1. If your caret is at the beginning of a list item:

         * |MyList item

      pasting a new list item will put the new item before it:

         * New list item
         * My list item

  2. If your caret is at the end of a list item:

         * My list item|

     pasting a new list item will put the new item after it:

         * My list item
         * New list item

  3. If your caret is in the middle of an existing list item,
     say just after the word "list"

         * My list| item

     pasting would yield:

          * My list
          * New list item
          * item

This "extra" bullet is actually the result of #1.

Comment 4

[sujay]

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

Comment 5

[rubydoo123]

users will perceive the added bullet as 

reviewed and approved

Comment 6

[Joe Francis]

see patch for this and other bugs attached to
bug 83918

Comment 7

[Blake Ross]

Missed 0.9.3.

Comment 8

[rubydoo123]

clearing the nsbranch keyword so as not to be confused with the current set of 
nsbranch bugs.

Comment 9

[Joe Francis]

fix checked in

Comment 10

[sujay]

verified in 8/31 build

however there is a minor caret issue after pasting the text
into the bullet...will file a new bug on this...