[$250] Web - IOU Participants - Nothing happens when press CMD+Enter

[kbecciv]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Issue was found when executed PR: #7702

Action Performed:

1. Go to NewDot
2. Click/tap on the green + icon on the home screen.
3. Click/tap on Request Money
4. Enter an amount and press Next
5. Enter a search query to get a reasonable-length list of results.
6. Use the arrow keys to highlight different result.
7. Press CMD+Enter.

Expected Result:

After Press CMD+Enter it should take you to the next page with your new selected user.

Actual Result:

When Press CMD+Enter nothing happens.

Workaround:

Unknown

Platform:

Where is this issue occurring?

• Web
• iOS
• Android
• Desktop App
• Mobile Web

Version Number: 1.1.79.2

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers): rybkina+061322@gmail.com/Qwerty123

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:

Bug5628527_step9_.mp4

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

[melvin-bot]

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

[rushatgabhane]

This is the expected UX for lists with untoggleable items

cc @roryabraham thoughts?

[sketchydroide]

I'm adding a deploy blocker has this seems to block the QA of #7702 and therefor blocking the deployment

[roryabraham]

This is the expected UX for lists with untoggleable items

Agree, this is how it was designed so I wouldn't say it counts as a regression or deploy blocker. If we want to make CMD+Enter work just like Enter for lists that don't support multiple selections, that's a new feature. It seems like a good idea to me but can be treated as separate

[roryabraham]

This can also be external

[rushatgabhane]

@roryabraham what problem is this feature request trying to solve?

Is it something like other similar pages allow cmd+enter to go next, so we should implement that pattern here too?

[roryabraham]

Is it something like other similar pages allow cmd+enter to go next, so we should implement that pattern here too?

Yeah, exactly

[madmax330]

Alright passing this on as external, thanks @roryabraham

[melvin-bot]

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

[laurenreidexpensify]

https://www.upwork.com/jobs/~01a590c870e2d44999

[melvin-bot]

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

[melvin-bot]

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

[roryabraham]

I'm actually going to just fix this myself because it's come up a few times and it will be a really quick and easy fix

[roryabraham]

@laurenreidexpensify I took down the upwork job for this

[roryabraham]

Got a PR up