Skip to content

Improvement Payment options Padding#6196

Merged
deetergp merged 1 commit into
Expensify:mainfrom
Santhosh-Sellavel:Improve_Payment_Options_Floating_Button
Nov 4, 2021
Merged

Improvement Payment options Padding#6196
deetergp merged 1 commit into
Expensify:mainfrom
Santhosh-Sellavel:Improve_Payment_Options_Floating_Button

Conversation

@Santhosh-Sellavel

Copy link
Copy Markdown
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

@deetergp deetergp left a comment

Copy link
Copy Markdown
Contributor

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

OSBotify commented Nov 4, 2021

Copy link
Copy Markdown
Contributor

✋ 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

OSBotify commented Nov 4, 2021

Copy link
Copy Markdown
Contributor

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

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

@parasharrajat

Copy link
Copy Markdown
Member

This PR is failing for #6232.

@OSBotify

Copy link
Copy Markdown
Contributor

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

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

@ogumen

ogumen commented Nov 21, 2021

Copy link
Copy Markdown

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