[$4000] Multiple weird issues on combination of expand message and scroll in chat message list page

[kavimuru]

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 any chat
2. Type some text and expand the message and open keyboard
3. Scroll the view
4. Type combinations of above 2

Expected Result:

Scrolling and expand should work properly

Actual Result:

1. There is blank background space below the compose box that shouldn't be there
2. The size of the compose box grows when scrolling though it shouldn't

Workaround:

unknown

Platforms:

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

• Android / Chrome
• iOS / Safari

Version Number: 1.2.54-2
Reproducible in staging?: Y
Reproducible in production?: Y
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: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

MXZE8121.2.MP4

ScreenRecording.4.mov

Expensify/Expensify Issue URL:
Issue reported by: @abdulrahuman5196
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1673596288476119

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01f84b7526ce4ad48b
• Upwork Job ID: 1615450817743364096
• Last Price Increase: 2023-01-31

[melvin-bot]

Triggered auto assignment to @puneetlath (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[puneetlath]

@kavimuru you mention "multiple weird issues on view positions and scrolling". Can you list what those issues are? From trying myself I see:

1. There is blank background space below the compose box
2. The size of the compose box grows when scrolling

Is there anything else?

[kavimuru]

@puneetlath scrolling issue and header also disappears when scrolling up, compose box is resizing when scrolling.

PKEQ2482.1.MP4

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

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

[eh2077]

It looks like duplicated, see #14161 (comment)

[melvin-bot]

@puneetlath, @sobitneupane Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[s77rt]

There is blank background space below the compose box that shouldn't be there

I think I saw an issue that reported that already and we decided to do nothing.

Header doesn't stay fixed when scrolling

This is already being handled here #13491
There is a possibility that this will be fixed after applying #13801 and #14392 However I don't think it will completely stay fixed as we are constantly applying marginTop style to the <ReportScreen /> (more details on the linked issue and PR)

[puneetlath]

It looks like duplicated, see #14161 (comment)

@eh2077 this seems similar to that issue, but not exactly the same.

I think I saw an issue that reported that already and we decided to do nothing.

Can you like to that @s77rt? It's not clear to me why we would choose to do nothing for this.

This is already being handled here #13491

Cool, thanks! I'll remove that from the scope of this issue.

[s77rt]

@puneetlath Sorry I don't remember what issue it was, maybe it was a different issue after all. FWIW there are many issues that are related to empty blank space that worth checking such as this one #13219 (although it's reported for mWeb not native, it's pretty similar)

[sobitneupane]

The size of the compose box grows when scrolling though it shouldn't

I was unable to reproduce this issue on IOS/Safari.

[puneetlath]

I just tried and I'm still able to reproduce this on iOS safari @sobitneupane.

[ArekChr]

I would like to investigate this issue, Callstack.

[melvin-bot]

📣 @ArekChr You have been assigned to this job by @mountiny!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

[ArekChr]

Hello,

About my update, I noticed this issue combines the other problems on mWeb with over-scroll behaviour.
I found a lot of similar issues related to unwanted padding between the keyboard and input that occur in many projects. This issue is likely associated with WebKit, related thread here: https://bugs.webkit.org/show_bug.cgi?id=141832 Unfortunately, I haven't been able to turn off the scroll completely from the CSS level.

The issue is very tricky. From the dev tools inspector, I noticed that spacing between the keyboard and input is not accessible when focusing on the entire HTML element, the screenshot above.

[MelvinBot]

@puneetlath @ArekChr @sobitneupane this issue is now 4 weeks old and preventing us from maintaining WAQ, can you:

• Decide whether any proposals currently meet our guidelines and can be approved as-is today
• If no proposals meet that standard, please take this issue internal and treat it as one of your highest priorities
• If you have any questions, don't hesitate to start a discussion in #expensify-open-source

Thanks!

[MelvinBot]

Current assignee @sobitneupane is eligible for the Internal assigner, not assigning anyone new.

[MelvinBot]

@puneetlath, @ArekChr, @sobitneupane Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[puneetlath]

I chatted with callstack in slack and they're going to see if they can pull in some other devs on this.

[puneetlath]

Callstack is hoping to have a sr dev look into this tomorrow.

[tjaniczek]

Hi @puneetlath , I'm looking into this since yesterday. So far with no success, unfortunately. I'll try out a couple of other ideas today as well. But with this being known iOS issue, chances of fixing this behaviour to full extend are bleak. Will keep you posted

[tjaniczek]

Hi @puneetlath , as the last effort I tried to find a workaround by applying absolute/fixed positioning to the input box but Safari randomly decided to slice app's drawing area (?) so this won't work either. I think I'll take Vit's advice and won't waste any more time on this, since I ran out of ideas. Hope that's ok!

Screen.Recording.2023-02-16.at.13.04.49.mov

[mountiny]

@tjaniczek I think this makes sense. @puneetlath Should we keep this open or close this if we are confident this is related to Webkit issue which is a bit out of our control?

[puneetlath]

Ugh, safari. Ok thank you for your investigation. I agree that it's probably best to close for now. I'm going to go ahead and close, but anyone feel free to reopen the issue if you disagree or if you think there's a solution!

[abdulrahuman5196]

@puneetlath @mountiny I am not sure on the process here. Does the reporter of the bug(me) gets compensated on this issue as it is a valid bug though.

[puneetlath]

In this case, no. We compensate for bugs that we decide to and are able to fix.