Closed
Bug 114928
Opened 23 years ago
Closed 23 years ago
hang/freeze after deleting list item using backspace key
Categories
(SeaMonkey :: Composer, defect, P1)
SeaMonkey
Composer
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.7
People
(Reporter: sujay, Assigned: mozeditor)
References
Details
(Whiteboard: EDITORBASE [QAHP]; regression; checked in on trunk; checked in on 097)
Attachments
(1 file)
|
719 bytes,
patch
|
hwaara
:
review+
sfraser_bugs
:
superreview+
|
Details | Diff | Splinter Review |
using 12/12 build of netscape
1) launch netscape
2) launch composer
3) insert a list like this:
* sdfsdfsdfsdf
* sdfsdfsdfsdf
* sdfsdfsdfsdfs <----place caret in front of this list item in step 4)
* sdfsdfsdfsdf
4) place caret at beginning of 3rd list line(in front of second list set)
5) hit backspace
composer freezes up....you have to kill the application in order to get out.
|
Assignee | |
Comment 1•23 years ago
|
||
I think I should try to get this into 097.
Severity: major → critical
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → mozilla0.9.7
reassign to Joe.
Assignee: syd → jfrancis
Status: ASSIGNED → NEW
Whiteboard: EDITORBASE [QAHP]
|
|
Assignee | |
Updated•23 years ago
|
Status: NEW → ASSIGNED
Whiteboard: EDITORBASE [QAHP] → EDITORBASE [QAHP]; today
|
|
Assignee | |
Comment 3•23 years ago
|
||
*** Bug 115286 has been marked as a duplicate of this bug. ***
|
|
Assignee | |
Comment 4•23 years ago
|
||
This one line patch corrects an editing error I made when making changes to the
original patch in response to review comments. Doh. Always retest vigorously
after changing your patch.
Note that this bug doesn't just apply to list items. See duplicate bug 115286
for another scenario when testing the fix.
|
|
Assignee | |
Comment 6•23 years ago
|
||
need r and sr. Simon, can you sr? I also need to get this into the 097
nomination process, however that works.
Whiteboard: EDITORBASE [QAHP]; today → EDITORBASE [QAHP]; regression; FIXINHAND; need r=, sr=
|
||
Comment 7•23 years ago
|
||
Comment on attachment 61820 [details] [diff] [review]
one line patch to nsHTMLEditRules.cpp
sr=sfraser
Attachment #61820 -
Flags: superreview+
|
|
Assignee | |
Updated•23 years ago
|
Whiteboard: EDITORBASE [QAHP]; regression; FIXINHAND; need r=, sr= → EDITORBASE [QAHP]; regression; FIXINHAND; need a=
|
||
Comment 8•23 years ago
|
||
Looks like there's no r= yet -- please get that and then check into branch and
trunk -- a=brendan@mozilla.org for 0.9.7.
/be
|
|
||
Updated•23 years ago
|
Blocks: 114455
Keywords: mozilla0.9.7+
|
||
Comment 9•23 years ago
|
||
Same issue as bug 114750?
|
|
Assignee | |
Updated•23 years ago
|
Whiteboard: EDITORBASE [QAHP]; regression; FIXINHAND; need a= → EDITORBASE [QAHP]; regression; FIXINHAND; need r=
|
||
Comment 10•23 years ago
|
||
Comment on attachment 61820 [details] [diff] [review]
one line patch to nsHTMLEditRules.cpp
r=hwaara
Attachment #61820 -
Flags: review+
|
|
Assignee | |
Comment 11•23 years ago
|
||
This is fixed.
Checked in on trunk.
checked in on MOZILLA_0_9_7_BRANCH.
QA: please also test the two dups of this bug when verifying: 114750, 115286
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Whiteboard: EDITORBASE [QAHP]; regression; FIXINHAND; need r= → EDITORBASE [QAHP]; regression; checked in on trunk; checked in on 097
|
|
||
Comment 12•23 years ago
|
||
*** Bug 114750 has been marked as a duplicate of this bug. ***
|
||
Updated•23 years ago
|
Keywords: fixed0.9.7
|
||
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•