[HOLD for payment 2024-07-22] [$250] Android - Report Fields - Cursor position jumps to the beginning of the name field

[lanitochka17]

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.4.0
Reproducible in staging?: Y
Reproducible in prouring regression testing, add the test name, ID and link from TestRail: N/A
Issue reported bduction?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team

Issue found when executing PR #44275

Action Performed:

1. Open new expensify app
2. Create a workspace
3. Enable report fields feature
4. Open report fields setting
5. Tap on add field
6. Tap on Name and add something and save it
7. Tap on Name again

Expected Result:

The cursor positions itself at the end of the text

Actual Result:

The cursor jumps to the beginning of the saved text

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

Bug6533194_1720104132387.Screen_Recording_20240704_173222_New_Expensify.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01ecace54576044ee5
• Upwork Job ID: 1808958862801435309
• Last Price Increase: 2024-07-04
• Automatic offers:
• shubham1206agra | Reviewer | 102994077
• dominictb | Contributor | 103082291
• abzokhattab | Contributor | 103091472

Issue Owner

Current Issue Owner: @garrettmknight

[melvin-bot]

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

[github-actions]

👋 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.

[lanitochka17]

@marcochavezf 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

[marcochavezf]

I don't consider it a deploy blocker, and can be fixed externally

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

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.

[mountiny]

I am managing this project so going to take over this issue to make sure it gets fixed. Thanks for helping!

[rezkiy37]

I've opened a PR (#45074) for review 🙂

[dominictb]

Hi @mountiny @rezkiy37 Do you think we need to re-use the useAutoFocusInput? This bug will happens if we use the input in modal so we can

1. Prevent the duplicated code
2. Prevent the future bug

[rezkiy37]

Just to clarify: the app does not use useAutoFocusInput for this scenario with the new report field name. It uses TextSelectorModal under the hood and this component handles focusing.

[abzokhattab]

Hello team, I think my solution was employed in the PR. Am I eligible for an offer here?

cc @mountiny

[mountiny]

$50 to @abzokhattab as the issue was labeled as Help Wanted and we have used their solution

[melvin-bot]

📣 @dominictb 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer 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 📖

[garrettmknight]

@dominictb updated offer sent to you for $50

[abzokhattab]

@garrettmknight I believe there may be a mistake in the offer assignment, can you please recheck this comment
Thank you for your consideration!

[melvin-bot]

📣 @abzokhattab 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer 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 📖

[garrettmknight]

Oops, @abzokhattab you're right. I'll drop the contract to pay $50 when the solution works on prod.

[rushatgabhane]

PR reviewed and merged

[garrettmknight]

@abzokhattab all paid out!

[garrettmknight]

Looks like this made it to prod on Monday. Just tested and it's fixed.

[garrettmknight]

Payment Summary:

• Reviewer: @rushatgabhane $250

@rezkiy37 @mountiny is there a regression test for this already?

[garrettmknight]

Dropping to weekly for payment

[shubham1206agra]

Payment at end of project. Please close this.