529071 - Message Menu -> Copy To and Move To don't show sub-folders (Mac only)

Status: VERIFIED FIXED

(Thunderbird :: Folder and Message Lists, defect)

Description

[Mark Banner (:standard8)]

Accessing the Copy To and Move To options from the main message menu shows the accounts but not any sub-folder menus.

The context menu works fine.

Comment 1

[Andrew Sutherland [:asuth] (he/him)]

Attachment: use cloneNode again but this time be shallow v1

So, the moral of the story is that cloneNode is most definitely the only way for this to work.

Comment 2

[Mark Banner (:standard8)]

Comment on attachment 412809 [details] [diff] [review]
use cloneNode again but this time be shallow v1

r/sr/a=Standard8

Comment 3

[Andrew Sutherland [:asuth] (he/him)]

fixed on trunk:
http://hg.mozilla.org/comm-central/rev/8f3704461d3b

fixed in comm-1.9.1:
http://hg.mozilla.org/releases/comm-1.9.1/rev/8d20d28bdca0

Comment 4

[Mark Banner (:standard8)]

Checked into relbranch for build 2 of RC1:
http://hg.mozilla.org/releases/comm-1.9.1/rev/1b4292bd528c

Comment 5

[Aldo Cugnini]

This problem also happens with the Windows version (Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.7) Gecko/20100111 Lightning/1.0b1 Thunderbird/3.0.1), running on XP.

Comment 6

[Aldo Cugnini]

Actually -- there is a difference from the original post -- Accessing the Copy To and Move To options from the main message menu (or by right-clicking an email listing) does NOT show any sub-menu listing the accounts to where the email may be copied of moved.  Only a tiny (about 4x4 pixels) portion of a submenu is shown.

Comment 7

[John M]

I am also seeing this on Windows 7. Neither the Message menu or the context menu shows any available folders.

Comment 8

[Dan Mosedale (:dmosedale, :dmose)]

For symptoms that are different, commenting in an existing bug unfortunately doesn't usually get tracked in the right way.  If you could file a new bug, that would be very helpful.  Thanks!

Comment 9

[[e-mail]]

Comment on attachment 412809 [details] [diff] [review]
use cloneNode again but this time be shallow v1

>+              var popup = this.cloneNode(false);
>               popup._parentFolder = folder;
>               popup.setAttribute("class", this.getAttribute("class"));
>               popup.setAttribute("type", this.getAttribute("type"));
>               popup.setAttribute("fileHereLabel",
>                                  this.getAttribute("fileHereLabel"));
>               popup.setAttribute("showFileHereLabel",
>                                  this.getAttribute("showFileHereLabel"));
>               popup.setAttribute("oncommand",
Of course, now that you're cloning the node, you get all the attributes for free ;-)