1489629 - Adjust the size of the error images, sizing and position of text, and add footer branding to more views

Status: VERIFIED FIXED

(Firefox :: WebPayments UI, enhancement, P1)

Description

[Jared Wein [:jaws] (please needinfo? me)]

From https://bugzilla.mozilla.org/show_bug.cgi?id=1487395#c8,
> 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?

Comment 1

[Jared Wein [:jaws] (please needinfo? me)]

Attachment: Bug 1489629 - Adjust the size of the error images, sizing and position of text, and add footer branding to the error pages. r?sfoster

Comment 2

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

Comment on attachment 9007362 [details]
Bug 1489629 - Adjust the size of the error images, sizing and position of text, and add footer branding to the error pages. r?sfoster

Sam Foster [:sfoster] has approved the revision.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2b24a0fca9b7
Adjust the size of the error images, sizing and position of text, and add footer branding to the error pages. r=sfoster

Comment 4

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/2b24a0fca9b7

Comment 5

[Timea Cernea [:tbabos][inactive]]

Hi Jared,

The error images, sizing and the position of the text looks good now. However, on the payment Failure screen I have the feeling that is not centered correctly, check out: https://imgur.com/EpmKb4x

Comment 6

[Jared Wein [:jaws] (please needinfo? me)]

Hi Timea,

That error message is centered as good as we can do based on the long domain name which is wrapping to the next line. The text in the suggestions below the heading are all one line and are not wrapping too soon.

Comment 7

[Timea Cernea [:tbabos][inactive]]

Thanks Jared,

Based on Comment 5 and Comment 6 I will close this issue as Verified - Fixed.