127938 - Can't create folder from 3 pane

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect, P1)

Description

[Eugene Savitsky]

Searched thrue bugzilla but couldn't find this bug.

1. Open mail.
2. Crate account.
3. In 3 pane window click right button on a folder and press "New Folder...".
4. Enter name and press OK.
5. Dialog dissapears, but no folder was created.

This also happens from File->New->Mew Folder. Same with Subfolders.

A way to create a folder is only from filter menu.

moz 2002022603

PS It's my first time I use mozilla's mail for mailing from my 3 accounts (I
have my main computer killed :( ).

Comment 1

[laurel]

Main menu item and context menu, IMAP or POP -- no folder created.  Using feb26
commercial trunk.

Comment 2

[laurel]

urgent

Comment 3

[laurel]

Cerate folder is working from filter ui, so that is a workaround until 3-pane
new folder works.

Comment 4

[Navin Gupta]

cc mstoltz, his checkins look suspicious after reading checkin comments. 
We get this js error 

JavaScript error:
chrome://global/content/dialogOverlay.js line 70: opener has no properties

Comment 5

[Navin Gupta]

backing out mstoltz checkins from last night fixes the filter and new 
folder creation problem in my tree. 

Comment 6

[Asa Dotzler [:asa]]

I think we back mstoltz out unless we can fix this really quickly. We're in the
very late stages of 0.9.9 and we can't ship with this regression. 

Comment 7

[Mitchell Stoltz (not reading bugmail)]

I'll have a fix by the end of today - If I don't I'll back out.

Comment 8

[laurel]

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

Comment 9

[Mitchell Stoltz (not reading bugmail)]

Reassigning to me, I've got a fix.

Comment 10

[Mitchell Stoltz (not reading bugmail)]

Attachment: Fix - don't block access to opener from chrome scripts

Comment 11

[Mitchell Stoltz (not reading bugmail)]

Attachment: Complete patch

Comment 12

[Johnny Stenback (:jst)]

Comment on attachment 71721 [details] [diff] [review]
Complete patch

sr=jst

Comment 13

[Stuart Parmenter]

Comment on attachment 71721 [details] [diff] [review]
Complete patch

r=pavlov

Comment 14

[Daniel (Leaf) Nunes]

Comment on attachment 71721 [details] [diff] [review]
Complete patch

a=leaf on behalf of drivers.

Comment 15

[Mitchell Stoltz (not reading bugmail)]

Fix checked in.

Comment 16

[David :Bienvenu]

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

Comment 17

[David :Bienvenu]

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

Comment 18

[Matthias Versen [:Matti]]

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

Comment 19

[laurel]

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

Comment 20

[Karen Huang]

Verified on Windows 05-12-20-pr1
Linux 05-12-23-pr1
Mac 05-12-17-pr1