Closed Bug 80880 Opened 23 years ago Closed 23 years ago

Deleting in empty list items can misbehave

Categories

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

defect

Tracking

()

VERIFIED FIXED
mozilla0.9.2

People

(Reporter: mozeditor, Assigned: mozeditor)

References

Details

(Whiteboard: [list] fixed, reviewed, a=asa)

Kathy Brade said:

1 Open Composer
2 Add a few items to a bulleted list
3 Keep the last item blank (do not enter any text and just leave the bullet   
there)
4 Save the File and Reopen in Composer
5 Now, click with mouse in the last (empty) bullet item
6 Press the BACKSPACE key on keyboard (expecting that the bullet item gets   
deleted and cursor goes to the earlier item's last character)
7 Observe that as soon as you hit Backspace, the last item shifts down by a line 
and cursor stays there. Pressing Backspace again brings the last item back to 
where it was and moves the caret to the last-but-one item's first character.

Then Shrirang Khanzode said on 2001-05-14 22:59:
Oh God ! This is showing something different now (0514 trunk on windoz). I used 
kathy's testcase(2001-02-19 08:43). What I see is..after step6 , hitting 
backspace once does nothing..hitting it again does nothing..then hitting 
backspace the third time moves the caret to the earlier item (keeping the last 
item intact).
moz 092
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.2
Whiteboard: [list]
this appears to be fixed by my patch to 46209.
Whiteboard: [list] → [list] fix in hand
Priority: -- → P2
+    if (rect.width) 
Would prefer
+    if (rect.width > 0) 
just to make it obvious that you didn't botch or make a typo and meant to compare 
a pointer to null.

otherwise, r/sr=sfraser
Whiteboard: [list] fix in hand → [list] fixed, reviewed, need a=
Blocks: 83989
a= asa@mozilla.org for checkin to the trunk.
(on behalf of drivers)
Whiteboard: [list] fixed, reviewed, need a= → [list] fixed, reviewed, a=asa
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
thisis fixed..on all platforms..but..I see another bug, that is bug 86927
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.