[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9pre) Gecko/2008061102 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4) 1. Open Properties for 'Local Folders'. (for example) 2. Select 'Copies & Folders' tree-item. 3. Select another tree-item. 3r. {{ Error: document.getElementById(accountPickerId).selectedItem is null Source File: chrome://messenger/content/am-copies.js Line: 270 }} Account Manager (seems to) works fine, except for this item. Workaround: close AM then reopen it...

What about if you select a different Folder before switching to another pane?

(In reply to comment #0) > 1. Open Properties for 'Local Folders'. (for example) > 2. Select 'Copies & Folders' tree-item. Actually, any mail/news account, other than LF which has no C&F tree-item ;-> ***** (In reply to comment #1) [Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9pre) Gecko/2008061202 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4) I can't select another folder in that pane: the boxes all display 'Local Folders' (or 'Sent' or 'Drafts'); and the (3+) drop-down list(s) is empty when it opens. [<-- root cause !!?] *** [Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9pre) Gecko/2008061203 Thunderbird/3.0a2pre] (nightly) (W2Ksp4) No problem with TB.

OK, that seems much more like what I expected. SeaMonkey doesn't ship mail-folder-binding.xml, so it just ends up with an empty menulist.

(In reply to comment #3) > SeaMonkey doesn't ship mail-folder-binding.xml <http://mxr.mozilla.org/mozilla/source/mail/base/content/mail-folder-bindings.xml> added by "Bug 413781 XBLify folder-selection menus". Is there a way to unbrake SM, until we port that bug ?

The only way to unbreak SM, without porting that bug would be to (1) restore the removed portions of msgFolderPickerOverlay.xul (2) #ifdef inclusion of that overlay and (3) #ifdef the js changes to am-copies. Given that the porting steps are (1) move mail-folder-bindings.xml to from mail/ to mailnews (2) add mail-folder-bindings.xml to SM's jar.mn, and (3) add http://mxr.mozilla.org/seamonkey/source/mail/base/content/messenger.css#262 to SM's messenger.css, that actually seems like the simpler solution to me.

(In reply to comment #5) > Given that the porting steps are (1) move mail-folder-bindings.xml to from > mail/ to mailnews Could you/someone do that (cvs copy/move, etc) ?

cvs moves require coordination with mozilla IT. Given the limited history/blame for the file, my vote would be to just cvs remove/add it in a standard patch. That's a decision for the peers though.

jminta's suggested solution in comment 5 sounds right to me. Coordination with IT just means "filing a bug", I think, so I guess I'd vote for that, assuming the turnaround on that sort of bug isn't too long. I could live with the cvs remove/add plan too.

Yes, we should strive / for comment 5.

> (Av1-TB) <jar.mn> update after bug 439058 You need to change SeaMonkey's respective jar.mn as well.

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.0.1pre) Gecko/2008061902 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4) Per comment 5 suggestion.

Comment on attachment 325859 [details] [diff] [review] (Av1-TB) <jar.mn> update after bug 439058 r=dmose

Av1-TB, updated with new/future file name.

Comment on attachment 327434 [details] [diff] [review] (Av1a-TB) <base/*/*> update after bug 439058 [Checkin: Comment 19] r=dmose

Comment on attachment 327434 [details] [diff] [review] (Av1a-TB) <base/*/*> update after bug 439058 [Checkin: Comment 19] Phil, I'm looking for a confirmation of previous DanM's r+. (DanM is not around at the moment.)

Comment on attachment 327434 [details] [diff] [review] (Av1a-TB) <base/*/*> update after bug 439058 [Checkin: Comment 19] Argh, silent bugzilla collision: {{ dmose@mozilla.org 2008-07-10 14:34:32 PDT Attachment #327434 [details] [diff] Flag review?(dmose@mozilla.org) review+ }}

Bv1-SM, updated with new file name.

Comment on attachment 327434 [details] [diff] [review] (Av1a-TB) <base/*/*> update after bug 439058 [Checkin: Comment 19] Landed this on CVS trunk aka 1.9.0. Leaving bug open until SM patch is in also.

Comment on attachment 329096 [details] [diff] [review] (Bv1a-SM) <mailnews/*> Port to SM >+ content/messenger/folderWidgets.xml) (base/resources/content/folderWidgets.xml) ^ Spurious close bracket. r+sr=me with that fixed.

Bv1a-SM, with comment 20 suggestion(s).

Comment on attachment 329166 [details] [diff] [review] (Bv1b-SM) <mailnews/*> Port to SM [Checkin: Comment 22] Landed this on CVS trunk aka 1.9.0.

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.0.2pre) Gecko/2008071202 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4) It fixed a third case (= Junk Settings) too: {{ Error: document.getElementById("actionAccountPopup").selectFolder is not a function Source File: chrome://messenger/content/am-junk.js Line: 60 }} V.Fixed