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

Pass translateX prop to TextInput component #6341

Merged
merged 1 commit into from
Nov 17, 2021

Conversation

Jag96
Copy link
Contributor

@Jag96 Jag96 commented Nov 17, 2021

cc @roryabraham

Details

This PR updates the ExpensiTextInput component to pass translateX to the child TextInput component, fixing an issue on Android where all inputs had extra space. Adding CP Staging label to fix deploy blocker.

Fixed Issues

$ #6321

Tests/QA

  1. Sign out of newdot and focus the sign in input, confirm there is no extra space at the beginning
  2. Sign into newdot and confirm other inputs do not have extra space at the beginning. The inputs in settings->change password is an example that doesn't pass translateX, so those also confirm there are no regressions.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web Android iOS
image image image

@Jag96 Jag96 requested review from roryabraham and a team November 17, 2021 00:07
@Jag96 Jag96 self-assigned this Nov 17, 2021
@github-actions
Copy link
Contributor

⚠️ ⚠️ Heads up! This pull request has the CP Staging label. ⚠️ ⚠️
Merging it will cause it to be immediately deployed to staging, even if the open StagingDeployCash deploy checklist is locked.

@roryabraham
Copy link
Contributor

We're going to use this PR to test #6199

@roryabraham roryabraham merged commit bde9be1 into main Nov 17, 2021
@roryabraham roryabraham deleted the joe-fix-input-translate branch November 17, 2021 04:23
github-actions bot pushed a commit that referenced this pull request Nov 17, 2021
github-actions bot pushed a commit that referenced this pull request Nov 17, 2021
@OSBotify
Copy link
Contributor

🚀 Cherry-picked to staging by @Jag96 in version: 1.1.15-8 🚀

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

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

@@ -198,6 +198,7 @@ class BaseExpensiTextInput extends Component {
onBlur={this.onBlur}
onChangeText={this.setValue}
onPressOut={this.props.onPress}
translateX={this.props.translateX}
Copy link
Member

Choose a reason for hiding this comment

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

@Jag96 Just want to know where did you get to know about this prop? It's not in docs...

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I only knew about it because we've used it in other parts of the app, I wasn't aware it was deprecated

Copy link
Member

Choose a reason for hiding this comment

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

Interesting.

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @AndrewGable in version: 1.1.15-15 🚀

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

@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @roryabraham in version: 1.1.15-18 🚀

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

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @roryabraham in version: 1.1.16-10 🚀

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.

4 participants