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

Done button is not present to close keyboard #2371 #2431

Merged
merged 4 commits into from
Apr 21, 2021

Conversation

rafidaky
Copy link
Contributor

@rafidaky rafidaky commented Apr 16, 2021

Details

IOS - Secondary log in - "Done" button is not present to close keyboard #2371 issue
Issue is solved for ios, all other platforms are not affected by this change.

Fixed Issues

Fixes #2371

Tests

QA Steps

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

web

Mobile Web

mobile web

Desktop

desktop

iOS

Screen Shot 2021-04-16 at 12 21 14 AM

Android

Screen Shot 2021-04-16 at 9 16 19 AM

@rafidaky rafidaky marked this pull request as ready for review April 16, 2021 06:57
@rafidaky rafidaky requested a review from a team as a code owner April 16, 2021 06:57
@MelvinBot MelvinBot requested review from pecanoro and removed request for a team April 16, 2021 06:57
@rafidaky
Copy link
Contributor Author

updated

@rafidaky rafidaky requested a review from tgolen April 21, 2021 12:41
Copy link
Contributor

@tgolen tgolen left a comment

Choose a reason for hiding this comment

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

Thanks for splitting those out into platform-specific components!

@rafidaky
Copy link
Contributor Author

all updated!

@rafidaky rafidaky requested a review from tgolen April 21, 2021 19:52
Copy link
Contributor

@tgolen tgolen left a comment

Choose a reason for hiding this comment

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

Thanks for those changes!

@tgolen tgolen merged commit 1eb1643 into Expensify:main Apr 21, 2021
@tgolen
Copy link
Contributor

tgolen commented Apr 21, 2021

Oops, sorry @pecanoro I didn't realize there was another reviewer. If Rocio finds any issues, please submit them in a new PR.

@OSBotify
Copy link
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.

@@ -0,0 +1,31 @@
/**
* This is a KeyboardAvoidingView only enabled for ios && disabled for all other platforms
Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm, is this one only for ios? What's the difference between this one and index.ios.js then?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

enabled prop is different, that way it only works on ios

@OSBotify
Copy link
Contributor

OSBotify commented May 8, 2021

🚀 Deployed to production in version: 1.0.39-5🚀

platform result
🤖 android 🤖 failure ❌
🖥 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.

[hold for payment 28 April] IOS - Secondary log in - "Done" button is not present to close keyboard
4 participants