-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[$500] Assign task - Marking the Task as Complete or incomplete does not focus the composer #27541
Comments
Triggered auto assignment to @strepanier03 ( |
Job added to Upwork: https://www.upwork.com/jobs/~01dda7cab075cd4ea4 |
Bug0 Triage Checklist (Main S/O)
|
Triggered auto assignment to @alexpensify ( |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @mananjadhav ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.Assign task - Marking the Task as Complete or incomplete does not focus the composer What is the root cause of that problem?We are missing focusing logic on Task Status changes. What changes do you think we should make in order to solve the problem?We should use ReportActionComposeFocusManager to focus on the composer when Task is reopened here: App/src/pages/home/HeaderView.js Line 119 in 76b6914
Implemetation:Import import ReportActionComposeFocusManager from '../../libs/ReportActionComposeFocusManager'; then here: onSelected: () => {
Task.reopenTask(props.report, title);
ReportActionComposeFocusManager.focus();
}, We should also add focus when task is reopened using
Result: Screencast.from.19-09-23.07.59.26.AM.IST.webmWhat alternative solutions did you explore? (Optional)xx |
Hi i want help with this |
📣 @poonam286! 📣
|
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
I am able to recreate this but I raised here to discuss expected behavior. I get that we're not focusing the composer but why would we expect the composer needs to be focused after marking a task complete or incomplete? |
@mananjadhav Hi, I have updated the link and added implementation details in proposal(comment). |
Hi @strepanier03, we refocus on the TextInput elements whenever we expect the user to type something, such as opening a chat, start filling a form, navigating back to a report. With this we decrease friction of the user by intelligently focusing on the important areas of the App on user actions. |
I'll reopen if we decide to make a change. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
Expected Result:
Marking the Task as Complete or incomplete focuses the composer
Actual Result:
Marking the Task as Complete or incomplete does not focus the composer
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: v1.3.70-5
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
T146taskfocus-1.mp4
2023-09-15.14.17.08.mov
Expensify/Expensify Issue URL:
Issue reported by: @daveSeife
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1694057465609589
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: