249305 - nothing happens when clicking on "new folder" button from the "Move" dialog

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, defect)

Description

[Thomas L.]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040626 Firefox/0.9.1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040626 Firefox/0.9.1

When I go to the bookmark manager, then click on the "classify" button from the
tool bar, I obtain a dialog box. In that dialog, if I press the "new folder"
button, nothing happen at all.
Not yet implemented or bug?

Reproducible: Always
Steps to Reproduce:
1. Open bookmark manager
2. Click "classify" in the toolbar
3. Then select a branch in the tree and click "new folder" button

Actual Results:  
Nothing at all : that's the problem.

Expected Results:  
I expect that my favorite browser add a new node in the folder-tree on the left.

Same thing occur on Linux, with the same version and the same l10n (fr-FR WIP)
but I don't thing at all this could come from the l10n...

Comment 1

[Thomas L.]

Bug confirmed with the en-US.
Bugzilla database search for, without result.

Comment 2

[Thomas L.]

My apologies : the english term wasn't "Classify" but "Move".

Comment 3

[PikeUK]

I can't find a dupe either, confirming on:

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040701
Firefox/0.9.0+

Looks like this:

http://lxr.mozilla.org/aviarybranch/source/browser/components/bookmarks/content/addBookmark.js#291

needs to be updated to match this:

http://lxr.mozilla.org/aviarybranch/source/xpfe/components/bookmarks/resources/addBookmark.js#290

Comment 4

[Mike Connor [:mconnor]]

Pike, care to attach a patch based on that? :)

Comment 5

[PikeUK]

Attachment: Update createNewFolder

Here you go (can you believe I managed that much coding in just 42 hours)!

Comment 6

[Mike Connor [:mconnor]]

Comment on attachment 152288 [details] [diff] [review]
Update createNewFolder

42 hours is good enough for me! :)

will handle this in my checkin blitz... which is going to be a big one if/when
I escape my work project... grumble grumble

Comment 7

[Mike Connor [:mconnor]]

fixed, branch and trunk

thanks pike!

Comment 8

[Mike Connor [:mconnor]]

sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs, filter on "beltznerLovesGoats" to get rid of this mass change