Members - Pressing Enter for the first time mark user and moves to the next screen

[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!

---

Action Performed:

1. Open staging.new.expensify.com or start Desktop App
2. Log in with expensifail account
3. Create or go to any workspace
4. Open Members
5. Click Invite
6. Click 'Enter' key

Expected Result:

First user is selected and marked. The screen with the users list is still open

Actual Result:

First user is marked AND the NEXT screen opens

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

Reproducible in staging?: Yes

Reproducible in production?: No

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

Bug6102054_Web-Dektop-Members-List-Invite-1st-Enter.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

[OSBotify]

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

[melvin-bot]

Triggered auto assignment to @francoisl (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[francoisl]

I can reproduce the issue, looking into the cause now.

[francoisl]

I think this is working as intended, the difference between staging and production is that we autofocus the Name, email, or phone number text input:

Staging	Prod
staging.mov	prod.mov

If you press Enter while the text input is focused, we also select the current row and navigate back on production.
If you unfocus the search input on staging and press Enter, we also select the highlighted row.

I can't pinpoint to what change to make the input automatically focus now, but it feels like a better UX to me - when you open that invite page, you probably want to start typing a name/email/phone number to filter options, so it makes sense to autofocus the input.

[francoisl]

cc @luacmartins as discussed in Slack. Going to close this so we can move forward with the deploy.