App: [Hold for payment Aug 3] KeyboadShortcut is broken

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 Search with Cmd + K
  2. Open New Group with Cmd + Shift + K
  3. Click outside New Group popup to close it.
  4. Press Escape to close Search.

Expected Result:

Escape should close the Search page.

Actual Result:

Escape doesn’t close the Search page.

Workaround:

The user can click outside Search page/ click on the close button.

Platform:

Where is this issue occurring?

Web ✔️ iOS Android Desktop App ✔️ Mobile Web

Version Number: Logs: https://stackoverflow.com/c/expensify/questions/4856 Notes/Photos/Videos: Any additional supporting documentation

https://user-images.githubusercontent.com/29673073/125547056-43f4345e-824c-4abf-a4af-cb5493fe3737.mp4

Expensify/Expensify Issue URL:

View all open jobs on Upwork

Upwork job: https://www.upwork.com/jobs/~011663eb89a2b43197

About this issue

  • Original URL
  • State: closed
  • Created 3 years ago
  • Comments: 19 (15 by maintainers)

Most upvoted comments

@tgolen, @puneetlath, @rushatgabhane Eep! 4 days overdue now. Issues have feelings too…

OK, thanks! That should do the trick. 🟢 to work on this.

Posted to Upwork, assigning an engineer to review. https://www.upwork.com/jobs/~011663eb89a2b43197

Added Daily label so @puneetlath will see it for triage. @parasharrajat , can you elaborate what needs further planning? I saw @rushatgabhane 's Slack post but there are no comments there. We can followup in a comment there and ‘share with room’ to try to get more eyes on, if needed