Closed
Bug 80880
Opened 24 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•24 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
•