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

Security - Placeholder is overlapping with entered text #7422

Closed
kbecciv opened this issue Jan 26, 2022 · 24 comments
Closed

Security - Placeholder is overlapping with entered text #7422

kbecciv opened this issue Jan 26, 2022 · 24 comments
Assignees
Labels
Engineering External Added to denote the issue can be worked on by a contributor Monthly KSv2

Comments

@kbecciv
Copy link

kbecciv commented Jan 26, 2022

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Action Performed:

  1. Launch the app
  2. Log in with any account
  3. Go to Settings - Security - Close account
  4. On Enter email box start typing the email

Expected Result:

Placeholder is not overlapping with entered text

Actual Result:

Placeholder is overlapping with entered text

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • iOS (issue reproduce for small devices - iPhone 8)
  • Android (issue reproduce for small devices - Samsung Galaxy S21)

Version Number: 1.1.33 -2

Reproducible in staging?: Yes

Reproducible in production?: Yes

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Image.from.iOS.7.MP4

Expensify/Expensify Issue URL:

Issue reported by: Applause

Slack conversation:

View all open jobs on GitHub

@MelvinBot
Copy link

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

@bondydaa
Copy link
Contributor

well that's an annoying regression 😞

I would guess external can handle this though we should probably check to see what PR caused this.

@bondydaa bondydaa added the External Added to denote the issue can be worked on by a contributor label Jan 26, 2022
@MelvinBot
Copy link

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

@bondydaa bondydaa removed their assignment Jan 26, 2022
@mateusbra
Copy link
Contributor

mateusbra commented Jan 26, 2022

@parasharrajat is that an issue you're working on within your TextInput's PRs? if you're not working on this one I would like to.

@parasharrajat
Copy link
Member

parasharrajat commented Jan 26, 2022

Sorry but this is a duplicate of #7379.

@mateusbra
Copy link
Contributor

No worries, thank you for the reply.

@mvtglobally
Copy link

mvtglobally commented Jan 26, 2022

@mateusbra @parasharrajat
I do not believe #7379 is same. This issue is about text OVER placeholder. 7379 is only about how placeholder text displayed. Check this out.

Screen Shot 2022-01-26 at 4 38 38 PM

@parasharrajat
Copy link
Member

@mvtglobally the root cause is the same. Due to the long label, it is overlapping on the value.

@mvtglobally
Copy link

mvtglobally commented Jan 27, 2022

label should disappear like on bunch of other CTAs across Expensify.
If this text meant to persist, it should be not inside the box but somewhere outside...

Recording.384.mp4

@parasharrajat
Copy link
Member

parasharrajat commented Jan 27, 2022

Ah, you may be confusing two things here. Let me clarify this and then we can evaluate.

  1. The related field for this issue is using a label.
  2. Placeholder should vanish when the user starts typing.
  3. label should stack top in New Expensify when the user starts typing.
  4. Here label is bigger than the width of the Input box thus it wraps in the next lines. But Input is designed to have only one line label text.
  5. Thus the typed value is overlapping with the label.

In the other issue, I was reporting the same behavior but I used more technical statement which may be confusing here.

But we can keep this issue open and wait to see if this is fixed by #7379.

@MelvinBot
Copy link

This issue has not been updated in over 14 days. @kevinksullivan eroding to Weekly issue.

4 similar comments
@MelvinBot
Copy link

This issue has not been updated in over 14 days. @kevinksullivan eroding to Weekly issue.

@MelvinBot
Copy link

This issue has not been updated in over 14 days. @kevinksullivan eroding to Weekly issue.

@MelvinBot
Copy link

This issue has not been updated in over 14 days. @kevinksullivan eroding to Weekly issue.

@MelvinBot
Copy link

This issue has not been updated in over 14 days. @kevinksullivan eroding to Weekly issue.

@MelvinBot MelvinBot added the Weekly KSv2 label Feb 14, 2022
@MelvinBot
Copy link

This issue has not been updated in over 14 days. @kevinksullivan eroding to Weekly issue.

3 similar comments
@MelvinBot
Copy link

This issue has not been updated in over 14 days. @kevinksullivan eroding to Weekly issue.

@MelvinBot
Copy link

This issue has not been updated in over 14 days. @kevinksullivan eroding to Weekly issue.

@MelvinBot
Copy link

This issue has not been updated in over 14 days. @kevinksullivan eroding to Weekly issue.

@MelvinBot MelvinBot removed the Overdue label Feb 14, 2022
@MelvinBot
Copy link

This issue has not been updated in over 14 days. @kevinksullivan eroding to Weekly issue.

@kevinksullivan
Copy link
Contributor

This is held on #7379 , not overdue.

@kevinksullivan kevinksullivan added Monthly KSv2 and removed Weekly KSv2 labels Feb 15, 2022
@mvtglobally
Copy link

Issue not reproducible during KI retests. (First week)

@mvtglobally
Copy link

Issue not reproducible during KI retests. (Second week)

@kevinksullivan
Copy link
Contributor

Nice, looks like this was fixed by the other PR all along then. closing it out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering External Added to denote the issue can be worked on by a contributor Monthly KSv2
Projects
None yet
Development

No branches or pull requests

7 participants