Closed Bug 568181 Opened 15 years ago Closed 14 years ago

Port bug 566910 'copy password' context menu item for password manager

Categories

(SeaMonkey :: UI Design, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1a3

People

(Reporter: neil, Assigned: InvisibleSmiley)

References

Details

Attachments

(1 file)

We share the JS backend code so we only need to copy the context menu popup.
Assignee: nobody → jh
Status: NEW → ASSIGNED
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.
Attachment #455828 - Flags: superreview?(neil)
Attachment #455828 - Flags: review?(neil)
Keywords: helpwanted
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.]
Attachment #455828 - Flags: superreview?(neil)
Attachment #455828 - Flags: superreview+
Attachment #455828 - Flags: review?(neil)
Attachment #455828 - Flags: review+
Attachment #455828 - Attachment description: patch → patch [Checkin: comment 3]
Severity: normal → enhancement
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: