(In reply to neil@parkwaycc.co.uk from Bug 1001071 comment #14) > This caused bug 1106225 for SeaMonkey because you moved the oncommand from > the button to the menupopup where it gets duplicated throughout the folder > menu and fires multiple times. (In reply to WADA from Bug 1106225 comment #21) > Minimum required change for this bug > > http://mxr.mozilla.org/comm-central/source/mail/base/content/SearchDialog. > xul#183 > 183 <button id="fileMessageButton" type="menu" > label="&moveButton.label;" > Add oncommand="MoveMessageInSearch(event.target) > 184 accesskey="&moveButton.accesskey;" > 185 observes="file_message_button"> > Change 186 <menupopup type="folder" showFileHereLabel="true" > mode="filing" /> > Delete 187 ... > 188 </button> >

Pushed to comm-central: http://hg.mozilla.org/comm-central/rev/85c52bd62394

Comment on attachment 8564745 [details] [diff] [review] Patch v1 move oncommand back to button [Approval Request Comment] Regression caused by (bug #): Bug 1001071 User impact if declined: Moving emails from Search Box Results to other folders creates duplicate emails and/or phantom mail Testing completed (on m-c, etc.): tested and baked on 2.36 Risk to taking this patch (and alternatives if risky): none bustage fix. String changes made by this patch: None.

(In reply to Philip Chee from comment #3) > Comment on attachment 8564745 [details] [diff] [review] > Patch v1 move oncommand back to button > > [Approval Request Comment] > Regression caused by (bug #): Bug 1001071 > User impact if declined: Moving emails from Search Box Results to other > folders creates duplicate emails and/or phantom mail > Testing completed (on m-c, etc.): tested and baked on 2.36 > Risk to taking this patch (and alternatives if risky): none bustage fix. > String changes made by this patch: None. Flags: approval-comm-release?