Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improvement Payment options Padding #6196

Conversation

Santhosh-Sellavel
Copy link
Collaborator

Details

Fixed Issues

$ #5906

Tests & QA Steps

  1. Go to Settings -> Payments
  2. Click on Add Payment method
  3. In Web / Desktop, the payment options width will be increased now.

Tested On

  • Web
  • Desktop

To Ensure Any UI Breaks - Tests well!

  • Mobile Web
  • iOS
  • Android

Screenshots

Web

Screenshot 2021-11-04 at 2 03 07 AM

Desktop

Screenshot 2021-11-04 at 2 17 24 AM

iOS

Simulator Screen Shot - iPhone 12 - 2021-11-04 at 02 12 43

Mobile Web

Simulator Screen Shot - iPhone 12 - 2021-11-04 at 02 11 53

Android

Screenshot_1635972224

@Santhosh-Sellavel Santhosh-Sellavel requested a review from a team as a code owner November 3, 2021 20:50
@MelvinBot MelvinBot requested review from deetergp and removed request for a team November 3, 2021 20:50
Copy link
Contributor

@deetergp deetergp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@deetergp deetergp merged commit fa83e4c into Expensify:main Nov 4, 2021
@OSBotify
Copy link
Contributor

OSBotify commented Nov 4, 2021

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify
Copy link
Contributor

OSBotify commented Nov 4, 2021

🚀 Deployed to staging by @deetergp in version: 1.1.13-3 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@parasharrajat
Copy link
Member

This PR is failing for #6232.

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @Jag96 in version: 1.1.14-4 🚀

platform result
🤖 android 🤖 failure ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@ogumen
Copy link

ogumen commented Nov 21, 2021

Accessibility issues found in this PR:
Web / mWeb

  1. The visual headings Payments and Payment methods are not programmatically implemented as headings - failure of WCAG SC 1.3.1, similar to [low] JAWS+Chrome: Many Pages: Visual headings are not announced as such #5458
    Payments_The visual headings are not implemented semantically as headings
  2. The grey buttons on white background do not meet the minimum color contrast requirements of 3.00:1. The buttons use grey #C6C9CA which gives a color contrast ratio of 1.66:1 against white #FFFFFF - failure of WCAG SC 1.4.11, similar to [med] Color Contrast: Many Pages: Grey icons against white background fail contrast requirements #5506
  3. The grey text "Account ending in..." does not meet the minimum contrast requirements of 4.50:1 for small text. The grey is #7D8B90 which gives a color contrast ratio of 3.51:1 against white #FFFFFF - failure of WCAG SC 1.4.3, similar to [low] Color Contrast: Many Pages: Grey text against white background fails contrast requirements #5503
    Payments_grey buttons and grey text on white fail color contrast requirements
  4. The expandable "Add payment method" button does not announce its button role or collapsed/expanded state - failure of WCAG SC 4.1.2, similar to issue [high] JAWS+Chrome: Licenses: The expandable sections are announced with no role or state #5502
    Payments_Add payment method element announced with no role or state
  5. The payment options are announced with no button role - failure of WCAG SC 4.1.2, similar to [med] JAWS+Chrome: Many Pages: Multiple elements announced without button role #5446
    Payments_Payment options announced without button role
    Web
  6. The focus outline is not clear for payment options - failure of WCAG SC 2.4.7, similar to [low] Keyboard Navigation: Payments: Visual focus indicator is not fully visible on several controls #5606
    https://user-images.githubusercontent.com/88733897/142780160-39571953-d852-44f9-84a4-147b0d190869.mp4
    Android
  7. The focus is not set to the first payment option in the bottom overlay once the "Add payment method" button is activated - failure of WCAG SC 2.4.3
    https://user-images.githubusercontent.com/88733897/142780266-5bb89a2c-7eb2-4313-b818-d44d491a7039.mp4
  8. The grey buttons on white background do not meet the minimum color contrast requirements of 3.00:1. The buttons use grey #C6C9CA which gives a color contrast ratio of 1.66:1 against white #FFFFFF - failure of WCAG SC 1.4.11
  9. The grey text "Account ending in..." does not meet the minimum contrast requirements of 4.50:1 for small text. The grey is #7D8B90 which gives a color contrast ratio of 3.51:1 against white #FFFFFF - failure of WCAG SC 1.4.3
    Android_Grey buttons and text on white background fail minimum contrast requirements

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants