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

Fix validation messages hidden behind keyboard #4174

Merged
merged 3 commits into from
Jul 26, 2021
Merged

Fix validation messages hidden behind keyboard #4174

merged 3 commits into from
Jul 26, 2021

Conversation

rdjuric
Copy link
Contributor

@rdjuric rdjuric commented Jul 22, 2021

Details

Wraps our sign in steps in a KeyboardAvoidingView, moves the error message to stay above our Button.

Fixed Issues

$ #4080

Tests

  1. Open the app without being logged
  2. Trigger a error message (wrong email on the first step, or wrong password in the second)
  3. The error shows up above the button

QA Steps

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

web.mov

Mobile Web

mWeb.mp4

Desktop

desktop.mov

iOS

iOS.mp4

Android

android.mp4

@rdjuric rdjuric marked this pull request as ready for review July 22, 2021 11:29
@rdjuric rdjuric requested a review from a team as a code owner July 22, 2021 11:29
@MelvinBot MelvinBot requested review from stitesExpensify and removed request for a team July 22, 2021 11:29
Copy link
Contributor

@stitesExpensify stitesExpensify left a comment

Choose a reason for hiding this comment

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

Nice!

@stitesExpensify
Copy link
Contributor

stitesExpensify commented Jul 23, 2021

Code looks great! I just watched all of the videos though and I noticed on web that when the error message pops up the image on the right side scrolls down, I don't think that should be happening 😬

@rdjuric
Copy link
Contributor Author

rdjuric commented Jul 23, 2021

Code looks great! I just watched all of the videos though and I noticed on web that when the error message pops up the image on the right side scrolls down, I don't think that should be happening 😬

The blue image on the first video? I looked HARD and can't see it scrolling down when the error appears 🧐

@stitesExpensify
Copy link
Contributor

I'm sorry that's my fault, it was actually on desktop. The image of the phone shifts down

@rdjuric
Copy link
Contributor Author

rdjuric commented Jul 24, 2021

Ah, I can see it now! This happens on the main branch too, it doesn't look related to this PR (I also didn't change any code in the SignInPageLayoutWide)

Here's a vid of it happening on the main branch. You can repro it by making the window smaller than the email/password form.

desktop.mov

@stitesExpensify
Copy link
Contributor

Ah, well if it's happening on main then we can just merge. Thanks!

@stitesExpensify stitesExpensify merged commit 57dac86 into Expensify:main Jul 26, 2021
@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.

@OSBotify
Copy link
Contributor

🚀 Deployed to staging in version: 1.0.80-3🚀

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

@OSBotify
Copy link
Contributor

🚀 Deployed to production in version: 1.0.81-4🚀

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