Page MenuHomePhabricator

Adyen Checkout: Further CSS tweaks
Closed, ResolvedPublic2 Estimated Story Points

Description

Different fields have slightly different heights:

Screenshot_2021-08-31 Make your donation now - Payments(1).png (1×1 px, 100 KB)

This looks worse when more address fields are required:

Screenshot_2021-08-31 Make your donation now - Payments.png (1×1 px, 104 KB)

Event Timeline

DStrine set the point value for this task to 2.Aug 31 2021, 7:41 PM

Change 719358 had a related patch set uploaded (by Damilare Adedoyin; author: Damilare Adedoyin):

[mediawiki/extensions/DonationInterface@master] Adyen Checkout: Further CSS tweaks

https://gerrit.wikimedia.org/r/719358

Change 719358 merged by jenkins-bot:

[mediawiki/extensions/DonationInterface@master] Adyen Checkout: Further CSS tweaks

https://gerrit.wikimedia.org/r/719358

Hey @Damilare, so as mentioned on IRC here is what I'm seeing when testing your patch on mobile using Chrome under Android.

Ingenico - State select text height

Screenshot_20210907-222444_Chrome.jpg (2×972 px, 331 KB)

and

Adyen Checkout - State select text height

Screenshot_20210907-222432_Chrome.jpg (2×972 px, 423 KB)

Note: I'm pretty sure the Adyen Checkout cc field bug, no space between that and email, is unrelated to your patch.

Change 719358 merged by jenkins-bot:

[mediawiki/extensions/DonationInterface@master] Adyen Checkout: Further CSS tweaks

https://gerrit.wikimedia.org/r/719358

I've reverted that merged commit.

Change 719514 had a related patch set uploaded (by Damilare Adedoyin; author: Damilare Adedoyin):

[mediawiki/extensions/DonationInterface@master] Adyen Checkout: Further CSS tweaks

https://gerrit.wikimedia.org/r/719514

Change 719514 merged by jenkins-bot:

[mediawiki/extensions/DonationInterface@master] Adyen Checkout: Further CSS tweaks

https://gerrit.wikimedia.org/r/719514