Support for <popup> is going away (bug 578322). <popup> has been obsoleted for some time and support was flaky (e.g. bug 571567).

Is it as easy at it seems, that you only need to change all "popup" to "menupopup" in all those xul files from the mxr url (like in bug 571567). If yes, than I can make patch.

(In reply to comment #3) > Is it as easy at it seems, that you only need to change all "popup" to > "menupopup" in all those xul files from the mxr url (like in bug 571567). If > yes, than I can make patch. Yes I believe it should be, see also bug 578440 for the suite/ version (you can always just try one or two first and see if it fixes the current popup menu issues).

I'm compiling the patch right now.

(In reply to comment #5) > I'm compiling the patch right now. Oh, I'am ready with my patch. Should I upload it anyway?

Standrad8 waht the requirements review wise for the parts in suite/ is asking your enough ?

Your patch look very similar to mine :-) The only difference is, you didn't realign the onpopup's under the "id", like: - <popup id="mailContext" - onpopupshowing="return fillMailContextMenu(event);" - onpopuphiding="mailContextOnPopupHiding(event);"> + <menupopup id="mailContext" + onpopupshowing="return fillMailContextMenu(event);" + onpopuphiding="mailContextOnPopupHiding(event);">

Comment on attachment 458328 [details] [diff] [review] patch fixing the issue This looks fine, but please fix the indentation in multiple places, as per comment 8.

(In reply to comment #10) > Comment on attachment 458328 [details] [diff] [review] > patch fixing the issue > > This looks fine, but please fix the indentation in multiple places, as per > comment 8. done

You missed a couple, but I fixed them on checkin: http://hg.mozilla.org/comm-central/rev/5e29e16c191b