1482808 - Clear the field-specific merchant error invalid state when the field has been edited

Status: VERIFIED FIXED

(Firefox :: WebPayments UI, defect, P1)

Description

[Matthew N. [:MattN]]

"Once the user begins to type into a field with an in-context error, the error will disappear. This will also work if a user deletes text within the field."

https://mozilla.invisionapp.com/share/YAFI31XR3KP#/screens/275361828

Right now https://dxr.mozilla.org/mozilla-central/rev/4e56a2f51ad739ca52046723448f3129a58f1666/browser/components/payments/test/browser/browser_shippingaddresschange_error.js#158 will hang after bug 1476345 disables the save button if there are invalid field because setCustomValidity is called with merchant-specific errors call but they are never cleared.

Comment 1

[(Away until May 24) Jared Wein [:jaws] (please needinfo? me)]

Attachment: Bug 1482808 - Clear the field-specific merchant error invalid state when the field has been edited. r?sfoster

Comment 2

[Sam Foster [:sfoster] (he/him)]

Comment on attachment 9006701 [details]
Bug 1482808 - Clear the field-specific merchant error invalid state when the field has been edited. r?sfoster

Sam Foster [:sfoster] has approved the revision.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f669be991db6
Clear the field-specific merchant error invalid state when the field has been edited. r=sfoster

Comment 4

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/f669be991db6

Comment 5

[Hani Yacoub]

Attachment: error.png

I'm not sure when the in-context error will be displayed, to try to verify when the error will disappear.

I accessed "https://rsolomakhin.github.io/pr/single/" (FTU), entered valid data in every field except "ZIP Code" field, no in-context error is displayed, only a red outline. (Please see screenshot attached)

Comment 6

[(Away until May 24) Jared Wein [:jaws] (please needinfo? me)]

This bug only focused on shipping address errors that are supplied by the merchant after the user changes their shipping address. I'm not sure of a website that implements this yet because it was just added to the spec.

The error text for the zip code will appear in a tooltip if you hover over the field. We should probably have that message show up in a popup the same way as the merchant specific address fields. I've filed bug 1490077 to display these the same.

Comment 7

[Timea Cernea [:tbabos]]

Since bug 1490077 introduced the field-specific errors, the red popup will disappear after the user edited the incorrect data.

The same behavior was observed for the merchant errors that appear on the summary page after editing the invalid Shipping address or saved CC. 

Is this check enough to cover this issue fix?

Comment 8

[(Away until May 24) Jared Wein [:jaws] (please needinfo? me)]

Yes, it is. Thanks.

Comment 9

[Timea Cernea [:tbabos]]

Thanks Jared!

Based on Comment 7 and Comment 8, I will mark this issues as Verified - Fixed  on Windows 10/7, Mac OS 10.13 and Ubuntu 16.04.