60125 - List Properties dialog remains on the screen

Status: VERIFIED FIXED

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

Description

[Srinivasa Rao Borusu]

From Bugzilla Helper:
User-Agent: Mozilla/4.73 [en] (WinNT; I)
BuildID:    2000101014

I tried to insert a unnumbered bullet and after inserting it I reopened the List 
Properties dialog and selected none under List type and clicked on the OK 
button, List properties dialog remains on the screen.

Reproducible: Always
Steps to Reproduce:
1. Open New Composer window.
2. Select Format->List->List properties
3. Select Bullet(Unnumbered) List as List Type and Solid circle as Bullet Style.
4. Click on OK button.
5. Again Select Format->List->List Properties
6. Now Select None under List Type
7. Click on OK button.

Actual Results:  List Properties dialog remains on the screen

Expected Results:  It should disappear from the screen and the solid circle in 
the composer window should go off.

Comment 1

[R.K.Aa.]

following steps to repro i get this error in console with linux sea build 20001114

JavaScript error:
 line 0: uncaught exception: [Exception... "Component returned failure code:
0x80004003 (NS_ERROR_INVALID_POINTER)
[nsIEditorShell.GetElementOrParentByTagName]"  nsresult: "0x80004003
(NS_ERROR_INVALID_POINTER)"  location: "JS frame ::
chrome://editor/content/EdListProps.js :: onOK :: line 333"  data: no]

After this, the dialog-box became a "ghost" - flat blue and outlined but no
content, and composer editing area froze. Menuline was still intact and i could
close composer.

Resolving as new, OS: All.

Comment 2

[R.K.Aa.]

*** Bug 60126 has been marked as a duplicate of this bug. ***

Comment 3

[rubydoo123]

reassign to jfrancis

Comment 4

[Joe Francis]

charlie, can you look into why GetElementOrParentByTagName() is failing?

Comment 5

[Charles Manske]

I'll look into it.

Comment 6

[Charles Manske]

Attachment: Fix for bug

Comment 7

[Robin Whittle]

I had a zombie List Properties dialog box when trying to make a list item no 
longer a list.  The window could be dragged around, leaving a trail of edges 
everywhere.  The program continued to work, including with the cursor in the 
middle of the box and its trails.

A demo of the bug is contained within:

  http://gair.firstpr.com.au/public-files/mozilla-bugs/mb-comp-space-3.html

This is with a nightly build, Win98, 2000112908.  It happened every time I 
tried.

Comment 8

[Charles Manske]

We can now remove a list from the List Properties dialog (that what was causing
the described behavior.)

Comment 9

[sujay]

verified in 1/17 build.