Closed Bug 1569341 Opened 5 years ago Closed 5 years ago

Hide launch/copy buttons in edit mode

Categories

(Toolkit :: Password Manager, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: ntim, Assigned: ntim)

References

Details

Attachments

(1 file)

No description provided.

In the future you should include rationale in comment 0, filing bugs with no comment 0 is discouraged. I'm wondering what the rationale is… it seems like they could be useful if they used the live value in the fields instead of the value from storage. e.g. to test if the typed URL is correct.

(In reply to Matthew N. [:MattN] (PM me if requests are blocking you) from comment #2)

In the future you should include rationale in comment 0, filing bugs with no comment 0 is discouraged. I'm wondering what the rationale is… it seems like they could be useful if they used the live value in the fields instead of the value from storage. e.g. to test if the typed URL is correct.

This is just done to match the UX spec: https://mozilla.invisionapp.com/share/EJRHBMEDKF8#/screens/372931541

Pushed by ntim.bugs@gmail.com: https://hg.mozilla.org/integration/autoland/rev/f89ffd06d951 Hide launch/copy buttons in edit mode. r=MattN
Pushed by apavel@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d80347b05879 Update test_login_item.html expectations to be correct. r=bustage-fix on a CLOSED TREE
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: