28456 - Removing bullets on a bulleted list of more than one deletes text

Status: VERIFIED FIXED

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

Description

[bijals (gone)]

Steps:
1) Launch Composer
2) Create three rows of lines with text
abc
abcd
abcde
3) Highlight the three rows and click the bullet icon
4) Unhighlight the selection and click on one of the rows
5) Highlight the 3 bullets and hit the bullet icon again

Actual Results: Either list is completely unformatted or you lose lines of text

Expected results: Bullets removed and you are back to having 3 lines of text

Build Date/Platform: 2/16/00-15 on NT

Comment 1

[bijals (gone)]

You also get caret's for each row below the bulleted list.  Basically, multiple 
carets on the screen.

Comment 2

[Kathleen :Brade]

reassign to jfrancis

Comment 3

[Akkana Peck]

This is working fine (I get the "expected result" in a 2/21 release build on
Linux.  Perhaps Joe fixed it with his last round of changes?

Tentatively marking M15 as part of editor bug triage, but Joe can move it up if
he thinks he has it fixed or can fix it in M14.

Comment 4

[leger]

If working, can you please mark WorksForMe?

Comment 5

[bijals (gone)]

It does not work all the time.  In fact, another side effect is that it keeps 
inserting a carriage return/break everytime you remove a bullet.

Comment 6

[bijals (gone)]

Okay, here is a reproduceable example.  

Steps:
1) Open a new mail message
2) Type "a" and return
3) Type "b" and return
4) Type "c" and return
5) Type "d" and return
6) Type "e" and return
7) Type "f" and return
8) Type "g" and return
9) Select rows with letters "a", "b", and "c"
10) Click the bullet icon
(Notice weird selection)
11) Click bullet icon again
(Weird behavior)
12) Click bullet icon again
(Some data lost and weird behavior)
13) Do this as well as remove bullets several times

Actual Results: Unexpected selections and bullet behavior

Expected Results: Only the 3 lines are bulleted and the bullet is removed by 
clicking the bullet icon the 2nd time.  Basically, apply and remove bullet 
behavior on the 3 lines.

Build Date/Platform: 2/21/00-08 on NT

Highly visible and common.

Comment 7

[bijals (gone)]

Sorry, describing weird behavior in above comments:

After completing step 10, the weird behavior is that either
1) "d" and "e" are also selected when they should not be
2) Blank line between "c" and "d" indicated by a blue caret
or
1) "a", "b", "c", and "d" are selected and in bullets
2) Carriage return between "d" and "e"

After completing step 11, nothing happens since you have to hit the bullet icon 
2 times to remove a bullet

After completing step 12, no more bullets, but the extra carriage return says 
between "d" and "e"

After hitting bullet icons several times, the blank rows between "d" and "e" 
increase and initial list is all over the place.

Comment 8

[Joe Francis]

accepting bug

Comment 9

[leger]

This is not a beta stopper as discribed.  Putting on PDT- radar for dogfood and 
beta1.

Comment 10

[Joe Francis]

this is ugly.  I've looked into this and the real problem is that selection is 
not properly preserved.  I knew we had trouble here with inline stlye, but I 
wasn't aware that the list code could also exhibit this problem until now.  

So the bad news is that it's hard to fix.  The good news is that when I put in 
the code to properly track selection for bold/italic/underline, I can also use 
that for lists as well.

Comment 11

[sujay]

adding EB in status summary...ender blocker...

Comment 12

[Joe Francis]

updating estimated date: original estimate was assuming that I was working on it 
from several days ago, when instead I have worked on a host of other worthy bugs.

Comment 13

[Kathleen :Brade]

remove PDT- and beta1/dogfood
move to M15

Comment 14

[rubydoo123]

list work set for M16

Comment 15

[rubydoo123]

updating keyword and status whiteboard to reflect that this is a beta 2 feature 
work bug that the Composer team deems a must fix for beta 2.

Comment 16

[Joe Francis]

fixed

Comment 17

[sujay]

will verify after 38991 gets fixed.

Comment 18

[sujay]

verified in 6/29 build.