Closed
Bug 1470184
Opened 7 years ago
Closed 6 years ago
Update the Preferences text and layout for Web Payments
Categories
(Firefox :: WebPayments UI, enhancement, P1)
Firefox
WebPayments UI
Tracking
()
VERIFIED
FIXED
Firefox 64
People
(Reporter: epang, Assigned: jaws)
References
Details
(Whiteboard: [webpayments] [user-testing])
Attachments
(2 files)
Update content in preferences
Reporter | ||
Updated•7 years ago
|
Assignee: nobody → jsavory
Updated•7 years ago
|
Status: NEW → ASSIGNED
Priority: P3 → P1
Updated•7 years ago
|
Flags: qe-verify-
Reporter | ||
Comment 1•6 years ago
|
||
Update to Firefox Preferences from ixd spec.
Flow can be found here:
https://mozilla.invisionapp.com/d/main#/console/13170472/288300821/preview
Reporter | ||
Updated•6 years ago
|
Assignee: jsavory → nobody
Status: ASSIGNED → NEW
Updated•6 years ago
|
Flags: qe-verify-
Priority: P1 → P3
Whiteboard: [webpayments] [ux] → [webpayments-reserve]
Updated•6 years ago
|
Flags: qe-verify+
QA Contact: hani.yacoub
Updated•6 years ago
|
Whiteboard: [webpayments-reserve] → [webpayments-reserve][ux]
Updated•6 years ago
|
Whiteboard: [webpayments-reserve][ux] → [webpayments-reserve]
Updated•6 years ago
|
Priority: P3 → P2
Whiteboard: [webpayments-reserve] → [webpayments-reserve] [user-testing]
Updated•6 years ago
|
Summary: Fix the Preferences text and layout → Update the Preferences text and layout for Web Payments
Updated•6 years ago
|
Whiteboard: [webpayments-reserve] [user-testing] → [webpayments] [user-testing]
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Priority: P2 → P1
Assignee | ||
Comment 2•6 years ago
|
||
The description text should get reviewed by Brian
Assignee | ||
Comment 3•6 years ago
|
||
Hi Brian, the attached patch introduces the following description to the Preferences for the "Forms & Autofill" section:
"Firefox saves your address and credit cards to help you checkout faster. You can edit, delete, and create new credit cards and addresses here."
This is mostly the same from the attached mockup, except that I removed the <product name> part. However, the Address autofill part is not always used for "checkout" so the string in my opinion isn't really what we want.
Flags: needinfo?(brjones)
Comment 4•6 years ago
|
||
Comment on attachment 9012255 [details]
Bug 1470184 - Update the Preferences text and layout for Web Payments. r?MattN
Matthew N. [:MattN] (PM me if requests are blocking you) has approved the revision.
Attachment #9012255 -
Flags: review+
Comment 5•6 years ago
|
||
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #3)
> Hi Brian, the attached patch introduces the following description to the
> Preferences for the "Forms & Autofill" section:
> "Firefox saves your address and credit cards to help you checkout faster.
> You can edit, delete, and create new credit cards and addresses here."
>
> This is mostly the same from the attached mockup, except that I removed the
> <product name> part. However, the Address autofill part is not always used
> for "checkout" so the string in my opinion isn't really what we want.
Jared, if I'm following, I think you're suggesting something like:
Firefox can save your address and credit card information. You can add, delete, and edit address and credit card information here.
Works for me!
Flags: needinfo?(brjones)
Comment 6•6 years ago
|
||
Comment on attachment 9012255 [details]
Bug 1470184 - Update the Preferences text and layout for Web Payments. r?MattN
Francesco Lodolo [:flod] has approved the revision.
Attachment #9012255 -
Flags: review+
Pushed by mozilla@noorenberghe.ca:
https://hg.mozilla.org/integration/autoland/rev/3de6e1582de9
Update the Preferences text and layout for Web Payments. r=flod,MattN
Comment 8•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Comment 9•6 years ago
|
||
Hi Eric,
Unfortunately, I cannot access this document to check out the new Preferences text and layout: https://mozilla.invisionapp.com/d/main#/console/13170472/288300821/preview
Can you please provide it differently so I can verify the fix?
Thanks
Flags: needinfo?(epang)
Comment 10•6 years ago
|
||
Yeah, that's a URL which requires auth. https://mozilla.invisionapp.com/share/ABFIO5F9NPZ#/screens/275361872 is the public URL.
Flags: needinfo?(epang)
Comment 11•6 years ago
|
||
Verified the Preferences text and layout based on design doc:https://mozilla.invisionapp.com/share/ABFIO5F9NPZ#/screens/275361872.
There are two differences that I would like to point out:
1. "Nightly can save your address and credit card information." description is displayed instead of the one mentioned in Comment 5
2. The "Save Credit Cards..." button is quite wider than the one that can be seen in the spec doc. Is this intended or not?
Here is a screenshot of how it looks now on all OS:
https://imgur.com/WQKdv1w
Flags: needinfo?(jaws)
Assignee | ||
Comment 12•6 years ago
|
||
(In reply to Timea Babos from comment #11)
> Verified the Preferences text and layout based on design
> doc:https://mozilla.invisionapp.com/share/ABFIO5F9NPZ#/screens/275361872.
>
> There are two differences that I would like to point out:
>
> 1. "Nightly can save your address and credit card information." description
> is displayed instead of the one mentioned in Comment 5
Yes, this string was further tweaked to remove the product name portion. The second sentence, "You can edit, delete, and create new credit cards and addresses here." was also removed since those types of operations are common elsewhere within preferences but don't have a similar description string. It felt out of place and redundant.
> 2. The "Save Credit Cards..." button is quite wider than the one that can be
> seen in the spec doc. Is this intended or not?
> Here is a screenshot of how it looks now on all OS:
> https://imgur.com/WQKdv1w
Your screenshot is the intended outcome. Some alignments that are made on the spec don't match the paddings that we place on buttons. Thanks for the questions :)
Flags: needinfo?(jaws)
Comment 13•6 years ago
|
||
Based on comment 12 I'll mark this bug as verified.
Updated•6 years ago
|
status-firefox63:
--- → wontfix
You need to log in
before you can comment on or make changes to this bug.
Description
•