-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[HOLD for payment 2024-03-07] [$500] Web - IOU - I can't navigate using the "Tab" key #36513
Comments
Job added to Upwork: https://www.upwork.com/jobs/~0144e80c4837a63c84 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @cubuspl42 ( |
Triggered auto assignment to @alexpensify ( |
We think that this bug might be related to #vip-vsp |
ProposalPlease re-state the problem that we are trying to solve in this issue.Can't navigate using the TAB key on the money request page. What is the root cause of that problem?In #35154, we intentionally disable the TAB to "fix" #26326. App/src/pages/iou/request/IOURequestStartPage.js Lines 91 to 102 in 6914b13
But this affected all money request page, including participant and confirmation page. What changes do you think we should make in order to solve the problem?We should unregister the tab listener when the screen is out focus. We can use
What alternative solutions did you explore? (Optional)This isn't the best solution, but just in case. We can store the navigation focus state to a ref and return early if the page is not focused anymore. |
Just to confirm, does your solution keep #26326 fixed? |
ProposalPlease re-state the problem that we are trying to solve in this issue.Can not navigate by tab key in request money page. What is the root cause of that problem?App/src/pages/iou/request/IOURequestStartPage.js Lines 91 to 102 in 6914b13
This change disable all tab actions in request money page (which wrapped by What changes do you think we should make in order to solve the problem?
<TopTab.Screen name={CONST.TAB_REQUEST.MANUAL}>
{({navigation}) => {
const state = navigation.getState();
if (state?.index === 0) return <IOURequestStepAmount route={route} />;
return (
<View style={visibility.hidden}>
<IOURequestStepAmount route={route} />;
</View>
);
}} What alternative solutions did you explore? (Optional)Above solution need to change all {React.Children.map(children, (child, i) => {
if (!React.isValidElement(child) || child.type !== TopTab.Screen || !isFunction(child.props.children)) {
return child;
}
return React.cloneElement(child, {
children: (props) => {
const state = props?.navigation?.getState();
if (state.index === i) {
return child.props.children(props);
}
return <View style={visibility.hidden}>{child.props.children(props)}</View>;
},
});
})} |
📣 @damadoko! 📣
|
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
@cubuspl42 yes |
@damadoko I'm confused by the behavior of the discussed tab component ( |
@cubuspl42 yes, it actually render non-active tabs to the DOM tree as #26326 (comment) mentioned. This bug only happens with
|
I decided to approve the proposal by @bernhardoj. Both proposals suggested to keep the C+ reviewed 🎀 👀 🎀 |
Triggered auto assignment to @mountiny, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
📣 @cubuspl42 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
📣 @bernhardoj 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
PR is ready cc: @cubuspl42 |
Update: The PR is waiting for @mountiny's review but I'll try to find a replacement. I think you are offline. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.45-6 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-03-07. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Awesome, looks like automation kicked in here. Heads up, I will be offline until Tuesday, March 5, 2024. I will not be actively watching over this GitHub during that period. If anything urgent is needed here, please ask for help in the Open Source Slack Room-- thanks! |
Alright, I've completed the payment summary here: For this Upwork Job: https://www.upwork.com/jobs/~0144e80c4837a63c84 Closing - Everyone has been paid via Upwork. |
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: 1.4.41.1
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
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
I should be able to use it
Actual Result:
I can't navigate using the "Tab" key. It works in task request flow
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6379431_1707920607011.bandicam_2024-02-14_05-45-36-762.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: