Closed kbecciv closed 1 year ago
Triggered auto assignment to @jliexpensify (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details.
Platforms
in OP are ✅)We want the AddPayPalMePage
inputs to work the same as other inputs
In AddPayPalMePage
we are using Button instead of the custom Form, and in Button we trigger onPress
prop when enter key is pressed, but we don't check for modifier keys (ctrl/cmd/shift etc) here. So enter
with any modifier key combination works.
We can do these things
Update button enter
key listener, and return early if any modifier key was pressed (this can be done via prop)
if (e.shiftKey || e.ctrlKey || e.altKey || e.metaKey) return
Migrate the AddPayPalMePage
to use the custom form component. Along with minor things, we would need to add a new onyx key, update the code to match the existing UI, and optionally add validation errors.
N/A
Is this still an issue? I just tested on Chrome on v1.3.46-0 and SHIFT+Enter does nothing for me.
@kbecciv can you retest and share a new video?
Unable to reproduce it on build 1.3.46-0
https://github.com/Expensify/App/assets/93399543/753ac3d1-5b14-4be7-89c3-3ac7796e162a
Nice one, thanks @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!
Action Performed:
Expected Result:
App should not trigger save on shift enter for paypal.me as it does not for similar type of single line description field in request money/ split bill
Actual Result:
App trigger save on shift enter for paypal.me
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.44-0 Reproducible in staging?: y Reproducible in production?: y 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
https://github.com/Expensify/App/assets/93399543/4593cc57-a942-43ed-9e47-557a86c42fe2
https://github.com/Expensify/App/assets/93399543/8db8d931-4045-4f9c-9460-df1a03e30625
Expensify/Expensify Issue URL: Issue reported by: @dhanashree-sawant Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1690171163554959
View all open jobs on GitHub