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

Main chat composer not visible on mobile web #8135

Closed
mvtglobally opened this issue Mar 14, 2022 · 9 comments
Closed

Main chat composer not visible on mobile web #8135

mvtglobally opened this issue Mar 14, 2022 · 9 comments
Assignees

Comments

@mvtglobally
Copy link

mvtglobally commented Mar 14, 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. Download and install the mobile app
  2. Sign in to New Expensify on mobile web. There will be a banner that says Open in the New Expensify App
  3. Open a DM.
  4. Tap on the compose box to focus it.

Expected Result:

You should be able to see and type in the composer.

Actual Result:

The composer is automatically scrolled out of view.

Workaround:

unknown

Platform:

Where is this issue occurring?

  • Mobile Web (iOS Safari)

Version Number: 1.1.42-3
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

RPReplay_Final1646964880.MP4

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

View all open jobs on GitHub

@MelvinBot
Copy link

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

@Justicea83 Justicea83 removed their assignment Mar 14, 2022
@Justicea83
Copy link
Contributor

Can't reproduce the error

@roryabraham roryabraham added the External Added to denote the issue can be worked on by a contributor label Mar 15, 2022
@MelvinBot
Copy link

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

@roryabraham roryabraham removed the External Added to denote the issue can be worked on by a contributor label Mar 15, 2022
@roryabraham
Copy link
Contributor

roryabraham commented Mar 15, 2022

Actually, I think we really just need to revert #6413, which never really worked as expected. More context in the PR and here

@roryabraham roryabraham added Weekly KSv2 and removed Daily KSv2 labels Mar 15, 2022
@thesahindia
Copy link
Member

@roryabraham, is this eligible for reporting? I had reported this earlier here.
Couldn't find any other report so don't know if it was reported before.

@roryabraham
Copy link
Contributor

Actually I think this is not eligible for reporting simply because it's a regression we identified a long time ago and forgot to fix. That's probably why @parasharrajat marked it as a duplicate in Slack. I'll handle the revert myself as soon as I have a few spare cycles.

@thesahindia
Copy link
Member

Ah cool, thanks for the clarification 😃 .

@roryabraham
Copy link
Contributor

Sorry, this is a pretty simple fix (in that it's just a revert for now), but I just haven't gotten around to it yet.

@roryabraham
Copy link
Contributor

I actually can't reproduce this either. Going to close it – feel free to reopen if it's reproducible.

@melvin-bot melvin-bot bot removed the Overdue label Apr 8, 2022
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

6 participants