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

Web - Chat - Scroll bar appears when adding emojis #2599

Closed
isagoico opened this issue Apr 27, 2021 · 8 comments
Closed

Web - Chat - Scroll bar appears when adding emojis #2599

isagoico opened this issue Apr 27, 2021 · 8 comments
Assignees

Comments

@isagoico
Copy link

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


Expected Result:

The emoji is added and text field should not have any visual changes

Actual Result:

There is a drop down scrolling in the text field

Action Performed:

  1. Go to staging.expensify.cash and login
  2. Select any user to initiate chat
  3. In the text input field add an emoji and observe the text field

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

Platform:

Where is this issue occurring?

Web ✔
iOS
Android
Desktop App
Mobile Web

Version Number: 1.0.32-0

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

Notes/Photos/Videos: Any additional supporting documentation

image

Expensify/Expensify Issue URL:

View all open jobs on Upwork

@isagoico isagoico added the AutoAssignerTriage Auto assign issues for triage to an available triage team member label Apr 27, 2021
@MelvinBot
Copy link

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

@MelvinBot MelvinBot removed the AutoAssignerTriage Auto assign issues for triage to an available triage team member label Apr 27, 2021
@SofiedeVreese
Copy link
Contributor

SofiedeVreese commented Apr 28, 2021

Hey @bfitzexpensify I don't think you have to create a separate issue for this. You can just triage the E/E.cash issue and assign to an engineer if you feel you've triaged it properly. I've closed the dupe issue as they were both E/E.cash issues!

That said, I'm actually not able to reproduce this on my end:

Screen.Recording.2021-04-29.at.8.59.59.am.mov

If you are able to reproduce then I recommend assigning to Engineering to continue the triage process and see if this is a good one for an external label. See https://stackoverflow.com/c/expensify/questions/8561 for more context!

@parasharrajat
Copy link
Member

@SofiedeVreese I think OP missed that the issue is coming on Windows.

@bfitzexpensify
Copy link
Contributor

@SofiedeVreese yes I just went into autopilot mode 😅. Ty for the clean-up!

I don't have a windows machine to test, but also cannot replicate this on Mac.

Tagging engineering to see if we have a way of virtually testing/reproducing this on Windows.

@MelvinBot
Copy link

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

@bfitzexpensify bfitzexpensify added the Weekly KSv2 label Apr 30, 2021
@jasperhuangg
Copy link
Contributor

This might be fixed by the line-height fix that will be going out to production soon. Can you check to see if this happens on staging? @isagoico

@isagoico
Copy link
Author

isagoico commented May 3, 2021

This was not reproducible in staging anymore.
image

@Dal-Papa
Copy link

Dal-Papa commented May 4, 2021

Sounds like the fix is about to come in production. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants