Skip to content

[No QA] Small design tweaks - #7373

Merged
stitesExpensify merged 2 commits into
Expensify:mainfrom
parasharrajat:trans-amount-scaffold
Jan 25, 2022
Merged

[No QA] Small design tweaks#7373
stitesExpensify merged 2 commits into
Expensify:mainfrom
parasharrajat:trans-amount-scaffold

Conversation

@parasharrajat

@parasharrajat parasharrajat commented Jan 25, 2022

Copy link
Copy Markdown
Member

Details

  1. Set Border width to 1px for Mode selection.
  2. Set the font size for Fee value to normal 15px.

Fixed Issues

$ #3922

Tests

  • Verify that no errors appear in the JS console

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web | Desktop | Mobile Web

image

iOS

Android

@stitesExpensify
stitesExpensify merged commit fc3e62c into Expensify:main Jan 25, 2022
@OSBotify

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.

@stitesExpensify

Copy link
Copy Markdown
Contributor

Oops I forgot to put the design label on here 😭 cc: @shawnborton is there anything else that needs to be changed here?

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @stitesExpensify in version: 1.1.33-0 🚀

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

@OSBotify

OSBotify commented Feb 1, 2022

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @roryabraham in version: 1.1.33-3 🚀

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

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.

3 participants