Use card network icons in manage credit cards list

VERIFIED FIXED in Firefox 64

Status

()

enhancement
P1
normal
VERIFIED FIXED
8 months ago
8 months ago

People

(Reporter: sfoster, Assigned: sfoster)

Tracking

unspecified
Firefox 64
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox64 verified)

Details

(Whiteboard: [webpayments])

Attachments

(1 attachment)

Assignee

Description

8 months ago
Bug 1477102 added icons for the subset of card networks that are supported by Web Payments (Basic Card). We can use these to add a more informative icon in the formautofill card management UI in preferences.
Assignee

Comment 2

8 months ago
There's not a good way to test the CSS gets applied as the backgroundImage is on a pseudo element. So I settled for just verifying the .brand class is toggled on the <select> as appropriate.
Assignee

Updated

8 months ago
Assignee: nobody → sfoster
Priority: -- → P1
Comment on attachment 9013421 [details]
Bug 1495530 - Use card icons in form autofill card preferences screen. r?MattN

Matthew N. [:MattN] (PM me if requests are blocking you) has approved the revision.
Attachment #9013421 - Flags: review+
Status: NEW → ASSIGNED
Whiteboard: [webpayments]
Assignee

Comment 4

8 months ago
Try run with the changes requested, on the Trunk as early Beta/uplift patches: https://treeherder.mozilla.org/#/jobs?repo=try&revision=c1fdbf0ea631d283dbb37eb6c668ce6f7ff94ba1

Comment 6

8 months ago
Pushed by sfoster@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/32cde67dd330
Use card icons in form autofill card preferences screen. r=MattN

Comment 7

8 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/32cde67dd330
Status: ASSIGNED → RESOLVED
Last Resolved: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Flags: qe-verify?
Flags: qe-verify? → qe-verify+
QA Contact: hani.yacoub

Comment 8

8 months ago
Verified as fixed on Firefox Nightly 64.0a1 on Windows 10 x 64, Windows 7 x32, Mac OS X 10.13 and Ubuntu 16.04 x64.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.