Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.98k stars 2.99k forks source link

Expense - Tab shift from manual-scan-distance is delayed improvement of existing solution #55081

Open sumo-slonik opened 2 weeks ago

sumo-slonik commented 2 weeks ago

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: 9.0.79-0 Reproducible in staging?: Yes Reproducible in production?: Yes Issue reported by: Applause Internal Team Device used: Redmi note 10s Android 13 App Component: Money Requests

Action Performed:

  1. Launch app in mweb and Android
  2. Tab fab-- create expense
  3. Swipe the page and note the tab shift from manual-scan-distance

Expected Result:

Tab shift from manual-scan-distance must be consistent in both mweb and Android.

Actual Result:

Tab shift from manual-scan-distance is slow in mweb but not in Android.

Workaround:

Unknown

Platforms:

Screens

https://github.com/user-attachments/assets/d4e72a78-404d-4528-bd4b-861563c9f174

Expected behavior is a smooth change of tabs.

This issue is folowup to : https://github.com/Expensify/App/issues/54638

melvin-bot[bot] commented 2 weeks ago

Triggered auto assignment to @garrettmknight (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.

sumo-slonik commented 2 weeks ago

I am Kuba Nowakowski from the expert agency SWM, and I have been delegated to carry out this task.

garrettmknight commented 1 week ago

Cool, go for it @sumo-slonik !

sumo-slonik commented 1 week ago

Here, I addressed what we want to do with this issue: https://github.com/Expensify/App/issues/54638#issuecomment-2595318167

melvin-bot[bot] commented 5 days ago

@garrettmknight, @sumo-slonik Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

garrettmknight commented 5 days ago

Looks like you're good to go from that other issue!

garrettmknight commented 2 days ago

@sumo-slonik do you have an ETA on this one?

melvin-bot[bot] commented 2 days ago

@garrettmknight @sumo-slonik 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!

sumo-slonik commented 1 day ago

@garrettmknight https://github.com/Expensify/App/pull/55347 The PR is ready and awaiting review.