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

[$250] iOS - Thread - Chat is scrolled up when opening message in thread #41532

Closed
m-natarajan opened this issue May 2, 2024 · 27 comments
Closed
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors

Comments

@m-natarajan
Copy link

m-natarajan commented May 2, 2024

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

Version Number: 1.4.70-0
Reproducible in staging?: y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @danielrvidal
Slack conversation: expensify.slack.com/archives/C049HHMV9SM/p1714137578957479

Action Performed:
Launch New Expensify app.
Go to chat.
Send a message.
Long tap on the message > Reply in thread.
Expected Result:
The chat will be scrolled down with keyboard open and compose box focussed

Actual Result:
The chat view is at the top in the thread and keyboard is not open

Workaround:
unknown

Platforms:
Which of our officially supported platforms is this issue occurring on?

Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
MacOS: Desktop
Screenshots/Videos
Add any screenshot/video evidence

Bug6469277_1714677182947.RPReplay_Final1714676988.mp4

View all open jobs on GitHub

Issue OwnerCurrent Issue Owner: @thesahindia
@m-natarajan m-natarajan added DeployBlockerCash This issue or pull request should block deployment Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels May 2, 2024
Copy link

melvin-bot bot commented May 2, 2024

Triggered auto assignment to @francoisl (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

Copy link

melvin-bot bot commented May 2, 2024

Triggered auto assignment to @kevinksullivan (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

Copy link
Contributor

github-actions bot commented May 2, 2024

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

@m-natarajan
Copy link
Author

m-natarajan commented May 2, 2024

@kevinksullivan FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

@francoisl
Copy link
Contributor

Something kind of similar is already happening on the production version, where the view doesn't scroll down after you send a new message. The keyboard not opening in this case looks is not reproducible on production though, let's see if we can identify the root cause.

Screen.Recording.2024-05-02.at.1.12.29.PM.mov

@francoisl francoisl added the External Added to denote the issue can be worked on by a contributor label May 2, 2024
@melvin-bot melvin-bot bot changed the title iOS - Thread - Chat is scrolled up when opening message in thread [$250] iOS - Thread - Chat is scrolled up when opening message in thread May 2, 2024
Copy link

melvin-bot bot commented May 2, 2024

Job added to Upwork: https://www.upwork.com/jobs/~01d93082b948cabfef

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label May 2, 2024
Copy link

melvin-bot bot commented May 2, 2024

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

@francoisl
Copy link
Contributor

Tested locally against the current production version 1.4.69-2 and the keyboard also doesn't open on new threads, so it doesn't appear to be a regression in the current checklist. I also can't consistently reproduce the issue where the initial thread view isn't scrolled, which makes it hard to identify when the issue started happening.

Going to demote for now and keep open as an external issue.

@francoisl francoisl added Daily KSv2 and removed DeployBlockerCash This issue or pull request should block deployment Hourly KSv2 labels May 3, 2024
@mosesbabu
Copy link

mosesbabu commented May 3, 2024

Thread - Chat is scrolled up when opening message in thread
After looking into the issue with the chat feature in the Expensify app, I've come up with a list of probable causes and some suggestions for fixing them.

  1. UI/UX Implementation: It seems like the design of our reply feature might be causing problems with scrolling and keyboard behavior. I'll review the design and make sure it handles these interactions properly. I'll also test it thoroughly to ensure it works well on different devices.
  2. Event Handling: I think there might be some issues with how our app responds to long-taps on messages to start a reply. I'll double-check the event handlers and make sure they're triggering the right actions.
  3. Keyboard Management: There could be some issues with how the keyboard is managed when someone starts a reply. I'll review the logic behind this and ensure the keyboard opens up correctly with the message box focused. I'll test it across different parts of the app to make sure it's consistent.
  4. Platform-Specific Behavior: It's possible that the problem is showing up differently depending on whether users are on Android, iOS, or the web. I'll test the chat feature on all supported platforms to identify any platform-specific issues. I'll then implement fixes or workarounds to ensure a consistent experience.
  5. Dependency Compatibility: there might also be some compatibility issues with the dependencies or libraries we're using in the app. I'll review them and make sure they're compatible with the platforms and versions supported. If needed, I'll update or replace any problematic dependencies.
    By addressing these potential causes and implementing the suggested fixes, I believe i can resolve the issue with the chat feature and provide a better experience for our users.

Copy link

melvin-bot bot commented May 3, 2024

📣 @mosesbabu! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

  1. Make sure you've read and understood the contributing guidelines.
  2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
    Screen Shot 2022-11-16 at 4 42 54 PM
    Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

@mosesbabu
Copy link

Contributor details
Your Expensify account email: mosesarara@gmail.com
Upwork Profile Link: https://www.upwork.com/freelancers/~019b1a49c108f99151

Copy link

melvin-bot bot commented May 3, 2024

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

@bernhardoj
Copy link
Contributor

@francoisl the keyboard doesn't open is being handled in #41402

@mosesbabu
Copy link

mosesbabu commented May 3, 2024 via email

@cupliz
Copy link

cupliz commented May 3, 2024

Contributor details
Your Expensify account email: iyan.cupliz@gmail.com
Upwork Profile Link: https://www.upwork.com/freelancers/~0134106ade17d77663

Copy link

melvin-bot bot commented May 3, 2024

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

Copy link

melvin-bot bot commented May 8, 2024

@francoisl, @kevinksullivan, @thesahindia Huh... This is 4 days overdue. Who can take care of this?

@melvin-bot melvin-bot bot added the Overdue label May 8, 2024
@mosesbabu
Copy link

mosesbabu commented May 8, 2024 via email

@mallenexpensify
Copy link
Contributor

@thesahindia , can you review the above and provide an update plz?

@kevinksullivan
Copy link
Contributor

Also @mosesbabu can you please use the proposal template format and repost your proposal?

@kevinksullivan
Copy link
Contributor

Not overdue melv.

Copy link

melvin-bot bot commented May 10, 2024

@francoisl, @kevinksullivan, @thesahindia 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

@francoisl
Copy link
Contributor

Can we confirm if this is still reproducible? It looks like it's fixed for me on v1.4.72-1

Screen.Recording.2024-05-10.at.3.11.19.PM.mov

@mvtglobally
Copy link

Issue not reproducible during KI retests. (First week)

Copy link

melvin-bot bot commented May 14, 2024

@francoisl, @kevinksullivan, @thesahindia 10 days overdue. Is anyone even seeing these? Hello?

@mosesbabu
Copy link

mosesbabu commented May 14, 2024 via email

@francoisl
Copy link
Contributor

Hi @mosesbabu, no the issue is not assigned, but for the future please review https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md to see how to make proposals. Thanks!

That said, the issue can't be reproduced anymore, so I'm going to close this.

@melvin-bot melvin-bot bot removed the Overdue label May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors
Projects
None yet
Development

No branches or pull requests

9 participants