Closed
Bug 80880
Opened 23 years ago
Closed 23 years ago
Deleting in empty list items can misbehave
Categories
(Core :: DOM: Editor, defect, P2)
Core
DOM: Editor
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).
Assignee | ||
Comment 1•23 years ago
|
||
moz 092
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.2
Updated•23 years ago
|
Whiteboard: [list]
Assignee | ||
Comment 2•23 years ago
|
||
this appears to be fixed by my patch to 46209.
Whiteboard: [list] → [list] fix in hand
Updated•23 years ago
|
Priority: -- → P2
Comment 3•23 years ago
|
||
+ 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
Updated•23 years ago
|
Whiteboard: [list] fix in hand → [list] fixed, reviewed, need a=
Comment 4•23 years ago
|
||
a= asa@mozilla.org for checkin to the trunk. (on behalf of drivers)
Updated•23 years ago
|
Whiteboard: [list] fixed, reviewed, need a= → [list] fixed, reviewed, a=asa
Assignee | ||
Comment 5•23 years ago
|
||
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 6•23 years ago
|
||
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.
Description
•