"Close" button should be displayed instead of "OK" button when payment failed

VERIFIED FIXED in Firefox 64

Status

()

P1
normal
VERIFIED FIXED
7 months ago
7 months ago

People

(Reporter: hani.yacoub, Assigned: jaws)

Tracking

63 Branch
Firefox 64
Points:
---

Firefox Tracking Flags

(firefox63 wontfix, firefox64 verified)

Details

(Whiteboard: [webpayments])

Attachments

(1 attachment)

(Reporter)

Description

7 months ago
Affected versions]: 
Nightly 63.0a1

[Affected platforms]:
Platforms: Windows 10 x 64, Mac OS X 10.13 and Ubuntu 18.04 x64.

[Preconditions]:
1. Set the pref dom.payments.request.enabled to "true";
2. Set the pref dom.payments.loglevel to “Debug” to increase the verbosity of console messages.
3. Make sure you have at least one Shipping Address and one payment method saved in your browser (Saved Cards).

[Steps to reproduce]:
1. Go to "https://bug1447777.bmoattachments.org/attachment.cgi?id=8988961" page and click on "Pay(fail)" or "Pay(Timeout)".
2. Select any payment method and click on "Pay" button.

[Expected result]:
- The Failed due to timeout screen is displayed as seen in the spec: https://mozilla.invisionapp.com/share/S5FXRY0D8TZ#/screens/304878933
- "OK" button should be replaced with "Close" button.

[Actual result]:
"OK" button is displayed instead of "Close" button.
(Reporter)

Updated

7 months ago
Flags: qe-verify+
QA Contact: hani.yacoub
Whiteboard: [webpayments] [triage]
(Reporter)

Comment 1

7 months ago
Based on the comment from https://bugzilla.mozilla.org/show_bug.cgi?id=1470207#c11 this bug will handle the strings that are displayed to tell the user what went wrong to match the spec(https://mozilla.invisionapp.com/share/S5FXRY0D8TZ#/screens/304878932).
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Priority: -- → P1
Whiteboard: [webpayments] [triage] → [webpayments]
The copy for the timeout case suggests 

"The most likely cause is a temporary connection hiccup. Open a new tab to check your network connection or click “Close” to try again." 

However, until the tab-modal dialog is implemented, this will be difficult for the user unless they already have a 2nd window open. Once the payment dialog is open, the user is prevented from interacting with the browser window behind. 

Jared and I discussed this and I think for now we should go ahead and land the existing copy as the plan is to prioritize the tab-modal dialog in M4. If we revise that plan we may want to revisit this.
Also, the text is a little misleading. When this error screen is shown we have already given up on waiting for the complete() call and the request promise has been resolved. So their only option at this point (retry() maybe changes this?) is to close the dialog and cross fingers it works next time.
Comment on attachment 9006657 [details]
Bug 1487395 - Update strings on error pages, as well as convert apostrophes to use 'smart-quotes'. r?sfoster

Sam Foster [:sfoster] has approved the revision.
Attachment #9006657 - Flags: review+

Comment 6

7 months ago
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e9d83a2e788a
Update strings on error pages, as well as convert apostrophes to use 'smart-quotes'. r=sfoster

Comment 7

7 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/e9d83a2e788a
Status: ASSIGNED → RESOLVED
Last Resolved: 7 months ago
status-firefox64: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64

Comment 8

7 months ago
The error strings, as well as the "Close" button, are looking good on both the Payment Timeout and Failure screen.
However, I would like to point out that the size of the Images, as well as the font of the strings, are different compared to the spec. Check: https://imgur.com/4Lbrlrt
Also, the "Nightly Checkout" is missing from the footer. Should all this be considered as concerns now or it will be handled later?
Flags: needinfo?(jaws)
Sigh... sorry about that. Yeah we should fix both of those. Can you file *another* follow-up for those?
Flags: needinfo?(jaws)
status-firefox63: affected → wontfix
(In reply to Timea Babos from comment #8)
> The error strings, as well as the "Close" button, are looking good on both
> the Payment Timeout and Failure screen.
> However, I would like to point out that the size of the Images, as well as
> the font of the strings, are different compared to the spec. Check:
> https://imgur.com/4Lbrlrt
> Also, the "Nightly Checkout" is missing from the footer. Should all this be
> considered as concerns now or it will be handled later?

(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #9)
> Sigh... sorry about that. Yeah we should fix both of those. Can you file
> *another* follow-up for those?

I filed https://bugzilla.mozilla.org/show_bug.cgi?id=1489629 for this.

Comment 11

7 months ago
Thanks Jared!

Verified - fixed on the latest Nightly 64.0a1 (2018-09-09) on Windows 10, Mac OS 10.13 and Ubuntu 18.04.
The "Close" button is displayed whenever a payment is failed.
Status: RESOLVED → VERIFIED
status-firefox64: fixed → verified
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.