[HOLD][$1000] Gap between edit message and keyboard on mWeb chrome

[kavimuru]

Held on #30912
Held on #33725

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. Open the app in android chrome
2. Open any report
3. Edit any message

Expected Result:

There should be no gap between keyboard and edit message like there isn't any gap in android app

Actual Result:

There is large gap between edit message and keyboard on mWeb chrome

Workaround:

unknown

Platforms:

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

• Android / native
• Android / Chrome
• iOS / native
• iOS / Safari
• MacOS / Chrome / Safari
• MacOS / Desktop

Version Number: 1.2.91-1
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: Any additional supporting documentation

gap.android.chrome.mp4

Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1680022112433229

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~016ea8064e577f077b
• Upwork Job ID: 1646253435450802176
• Last Price Increase: 2023-04-12

[MelvinBot]

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

[MelvinBot]

Bug0 Triage Checklist (Main S/O)

• This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
• This bug is not a duplicate report (check E/App issues and #expensify-bugs)
  • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
• This bug is reproducible using the reproduction steps in the OP. S/O
  • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
  • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
• This issue is filled out as thoroughly and clearly as possible
  • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
• I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

[MelvinBot]

@adelekennedy Eep! 4 days overdue now. Issues have feelings too...

[MelvinBot]

@adelekennedy 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

[MelvinBot]

@adelekennedy 10 days overdue. I'm getting more depressed than Marvin.

[MelvinBot]

@adelekennedy 10 days overdue. I'm getting more depressed than Marvin.

[adelekennedy]

@kavimuru I'm not able to reproduce this 😬 can you give it a try and let me know if you're seeing the gap?

[kavimuru]

I tagged @dhanashree-sawant in the slack

[dhanashree-sawant]

Hi @adelekennedy, I am still able to recreate on latest staging.

Untitled.mp4

[MelvinBot]

@adelekennedy this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

[MelvinBot]

Job added to Upwork: https://www.upwork.com/jobs/~016ea8064e577f077b

[MelvinBot]

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

[MelvinBot]

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

[MelvinBot]

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

[cristipaval]

@koko57 Great, thanks for taking care of this one. Hold is removed now.

[cristipaval]

@koko57 what's the status on this one?

[koko57]

@cristipaval Sorry, I didn't notice that you removed on hold! BaseInvertedFlatList is still not migrated to FlashList yet - I think we should wait for it to be migrated first.

[koko57]

@cristipaval I aksed my colleague that is working on the migration - there's no ticket yet for migrating the chat list to Flashlist, but it's planned. Let's put it on HOLD once again.

[cristipaval]

Alright, I'll do it. Do we have a tracking issue for the migration on which we held this one?

[koko57]

Unfortunately I couldn't find one - these two were closed #28902 #28178. Some of the migrations that are currently being worked on are under the performance project like this one: #30912

[cristipaval]

I see. I think we're good if we held this one on #30912

[cristipaval]

Thank you @koko57!

[cristipaval]

Removing the HOLD. @koko57, do you have some spare time to investigate this one again? 🙏

[koko57]

@cristipaval We were holding for some other issue, because the ticket for the main chat wasn't created back then 🙂 It was created recently, so now we need to hold on this one (#33725).

But if I have some spare time I'll see what's the current status of the issue - if anything changed in the meantime

[cristipaval]

Thanks @koko57!

[cristipaval]

Still on hold.

[cristipaval]

same

[adelekennedy]

coming from our updated process - I think we should close this issue based on our new standards