We share the JS backend code so we only need to copy the context menu popup.
Assignee: nobody → jh
Status: NEW → ASSIGNED
Created attachment 455828 [details] [diff] [review] patch [Checkin: comment 3] The strings are in Toolkit, too, as is the test. Made the change from http://hg.mozilla.org/mozilla-central/rev/0eed88315d8e in addition to http://hg.mozilla.org/mozilla-central/rev/d57d62799e35.
Comment on attachment 455828 [details] [diff] [review] patch [Checkin: comment 3] >+ onpopupshowing="UpdateCopyPassword()"> Nit: JS statements end in semicolons. >+ oncommand="CopyPassword()"/> Ditto. Please fix these two before checkin. >- onselect="SignonSelected();"> >+ onselect="SignonSelected();" >+ context="signonsTreeContextMenu"> [I don't mind so much keeping in sync with toolkit here, I just wanted to point out but they could have saved a delete by inserting instead of appending.]
Comment on attachment 455828 [details] [diff] [review] patch [Checkin: comment 3] http://hg.mozilla.org/comm-central/rev/7a4c37247aa5
Attachment #455828 - Attachment description: patch → patch [Checkin: comment 3]
Severity: normal → enhancement
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a3
You need to log in before you can comment on or make changes to this bug.