Chris Lyon Reporter
	Description • 23 years ago

Using build 2001110603 on Win2K

Steps to Reproduce:
1. Right click on Personal Toolbar and select New Folder
2. Press Ok in the Create New Folder dialog

Results:
A dialog labelled Alert with the following text: 
appears: "NC:PersonalToolbarFolder"
When hitting OK, another identical dialog appears.
The folder then gets created.

	Chris Lyon Reporter
	Comment 1 • 23 years ago

Sorry, this is not limited to the Personal Toolbar, similar alerts appear 
wherever you create a folder (sidebar, manage bookmarks window)

	André Dahlqvist
	Comment 2 • 23 years ago

Verifying with 2001-11-06-12 on Linux.

	André Dahlqvist
	Comment 3 • 23 years ago

Raising severity since this is too ugly to be included in 0.9.6.

	Chris Lyon Reporter
	Comment 4 • 23 years ago

Changing OS to All besed on André's comments.

	André Dahlqvist
	Comment 5 • 23 years ago

I've narrowed it down somewhat.

2001-11-01-21   OK
2001-11-05-22   BROKEN

There aren't any tar.gz builds inbetween these dates, so I can't narrow it down
further.

	Chris Lyon Reporter
	Comment 6 • 23 years ago

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

	Alex Vincent [:WeirdAl]
	Comment 7 • 23 years ago

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

	Ben Goodger (use ben at mozilla dot org for email)
	Comment 8 • 23 years ago

last of the bookmarks bugs...

	Chris Lyon Reporter
	Comment 9 • 23 years ago

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

	Chris Lyon Reporter
	Comment 10 • 23 years ago

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

	Alfonso Martinez
	Comment 11 • 23 years ago

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

	xyzzy
	Comment 12 • 23 years ago

OK, who checked in debug code?  ;)

I get two alerts doing this, the first seems to be the "current" URL entry, the
second seems to be the parent ID within the tree.  This is also happening for
New Separator, and sounds similar to bug 110572.

	Neil Pryde
	Comment 13 • 23 years ago

These two alerts are located in bookmarksOverlay.js:

http://lxr.mozilla.org/seamonkey/source/xpfe/components/bookmarks/resources/bookmarksOverlay.js#766
http://lxr.mozilla.org/seamonkey/source/xpfe/components/bookmarks/resources/bookmarksOverlay.js#783

These lines are inserted by Blaky Blake.

-Neil.

	Neil Pryde
	Comment 14 • 23 years ago

See also:

http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/xpfe/components/bookmarks/resources/bookmarksOverlay.js
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&root=/cvsroot&subdir=mozilla/xpfe/components/bookmarks/resources&command=DIFF_FRAMESET&file=bookmarksOverlay.js&rev2=1.29&rev1=1.28

Blake, please remove them a.s.a.p.

Thanks you,
-Neil.

	Neil Pryde
	Comment 15 • 23 years ago

Aha, bug 110572 will also be fixed when these alerts are removed.

	Neil Pryde
	Comment 16 • 23 years ago

"This is also happening for New Separator"

bookmarks.xul makes use of the same overlay file, bookmarksOverlay.js so this
will also be fixed after removing the alerts.

-Neil.

	Paul Chen Assignee
	Comment 17 • 23 years ago

this fix was checked in as part of fix for 102311, marking fixed

	Chris Lyon Reporter
	Comment 18 • 23 years ago

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

	Chris Lyon Reporter
	Comment 19 • 23 years ago

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

	André Dahlqvist
	Comment 20 • 23 years ago

Verified fixed with a CVS build from 2002-03-20-21 on Linux.