App icon indicating copy to clipboard operation
App copied to clipboard

[$500] Android - Add Phone number UI not consistent - reported by @gadhiyamanan

Open mvtglobally opened this issue 3 years ago • 31 comments

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Action Performed:

  1. Go to setting > profile > Add phone number
  2. click on password
  3. check send validation button

Expected Result:

Send validation button should not be hidden

Actual Result:

Send validation button is hidden

Platform:

  • Android

Version Number: 1.1.99-0 Reproducible in staging?: Y Reproducible in production?: Y Logs: https://stackoverflow.com/c/expensify/questions/4856 Notes/Photos/Videos: https://user-images.githubusercontent.com/43995119/190951528-cf5f7ee0-d6e4-43f6-8f4f-3c38f89d6ab7.mp4

Expensify/Expensify Issue URL: Issue reported by: @gadhiyamanan Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1661685271625409

View all open jobs on GitHub

mvtglobally avatar Sep 19 '22 04:09 mvtglobally

Triggered auto assignment to @davidcardoza (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

melvin-bot[bot] avatar Sep 19 '22 04:09 melvin-bot[bot]

I can't reproduce on mobile because I am running on iOS. The workflow looks like fine on iOS though. Either way, what's shown on Android should mirror what's shown in iOS IMG_BF83BBE593B2-1

davidcardoza avatar Sep 20 '22 16:09 davidcardoza

Triggered auto assignment to @cead22 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

melvin-bot[bot] avatar Sep 20 '22 16:09 melvin-bot[bot]

@cead22 Eep! 4 days overdue now. Issues have feelings too...

melvin-bot[bot] avatar Sep 26 '22 06:09 melvin-bot[bot]

This is by design. The button should be disabled because the password hasn't been entered

cead22 avatar Sep 26 '22 22:09 cead22

@davidcardoza if you don't understand the issue or can't reproduce it, ask for clarification before assigning to engineering, or simply close the issue

cead22 avatar Sep 26 '22 22:09 cead22

@cead22 in android, when you click on a phone number user can see the button but when you click on the password keyboard will hidden

gadhiyamanan avatar Sep 28 '22 05:09 gadhiyamanan

@gadhiyamanan can you double check if when you click password and the button appears hidden, if you can scroll down and see the button? I think it's possible the button isn't hidden, but rather, the keyboard we show for the password is taller than the one we show for the phone number, and it's covering the button, but the button isn't really disappearing.

If you can confirm the button is actually gone and you can't scroll down and see it, please reopen this issue

cead22 avatar Sep 28 '22 20:09 cead22

@cead22 not able to scroll down, first I need to close the keyboard

the button disappears only for the password field and is reproducible only on android

gadhiyamanan avatar Sep 29 '22 07:09 gadhiyamanan

Triggered auto assignment to @luacmartins (Demolition), see https://stackoverflow.com/c/expensify/questions/8099 for more details.

melvin-bot[bot] avatar Sep 29 '22 18:09 melvin-bot[bot]

@luacmartins Whoops! This issue is 2 days overdue. Let's get this updated quick!

melvin-bot[bot] avatar Oct 03 '22 06:10 melvin-bot[bot]

@cead22 I think an external contributor can work on fixing this bug. Would you be against removing the demolition label and making this external?

luacmartins avatar Oct 03 '22 20:10 luacmartins

Not at all, go for it

cead22 avatar Oct 04 '22 00:10 cead22

Triggered auto assignment to @stephanieelliott (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

melvin-bot[bot] avatar Oct 04 '22 14:10 melvin-bot[bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @Santhosh-Sellavel (External)

melvin-bot[bot] avatar Oct 04 '22 14:10 melvin-bot[bot]

Current assignee @luacmartins is eligible for the External assigner, not assigning anyone new.

melvin-bot[bot] avatar Oct 04 '22 14:10 melvin-bot[bot]

This is related to keyboard hiding content, @tgolen this should be held right?

Santhosh-Sellavel avatar Oct 05 '22 15:10 Santhosh-Sellavel

Yes! It is. I've tested this exact flow with some of my recent changes and I'm pretty certain I'll have it fixed.

tgolen avatar Oct 05 '22 20:10 tgolen

Putting this on HOLD for now.

tgolen avatar Oct 10 '22 17:10 tgolen

This is still on HOLD while I try to get https://github.com/Expensify/App/pull/11586 merged. That will then allow this to be opened up for proposals again.

tgolen avatar Oct 24 '22 13:10 tgolen

This is no longer on hold and can have proposals created for it with the new code.

tgolen avatar Oct 31 '22 16:10 tgolen

Triggered auto assignment to @abekkala (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

melvin-bot[bot] avatar Oct 31 '22 16:10 melvin-bot[bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @Santhosh-Sellavel (External)

melvin-bot[bot] avatar Oct 31 '22 16:10 melvin-bot[bot]

Current assignee @tgolen is eligible for the External assigner, not assigning anyone new.

melvin-bot[bot] avatar Oct 31 '22 16:10 melvin-bot[bot]

https://www.upwork.com/ab/applicants/1587772488630054912/job-details

https://www.upwork.com/jobs/~015814913e9779f5c9

abekkala avatar Nov 02 '22 11:11 abekkala

I will fix your layout to use only 'position:absolute'. And then it will change by keyboardaware events.

FilipRado avatar Nov 02 '22 12:11 FilipRado

@abekkala Given that we're still early and no proposals have been accepted, I'm assigning myself per the new BZ process: https://stackoverflow.com/c/expensify/questions/14597/14598#14598.

JmillsExpensify avatar Nov 02 '22 19:11 JmillsExpensify

@FilipRado Please go through our past issues and learn how to write a proper proposal.

Also checkout out our guidelines to understand our process

Santhosh-Sellavel avatar Nov 02 '22 20:11 Santhosh-Sellavel

Still waiting for proposals.

JmillsExpensify avatar Nov 06 '22 20:11 JmillsExpensify

Still waiting for proposals. Increasing the price to $500.

JmillsExpensify avatar Nov 08 '22 19:11 JmillsExpensify