-
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
[HOLD for payment 2025-02-04] [HOLD for payment 2025-01-24] [$250] Editing a comment: the cursor is jumping to the end of the message when we type in the middle of the text making it impossible to edit the text #54335
Comments
This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989 |
Triggered auto assignment to @stephanieelliott ( |
Seems related to the grammar extension that they're using. |
I do have Grammarly installed |
Can you try without that? |
I dont know how to write without that |
I cannot reproduce it now even with it on, so I will keep monitoring if it happens again |
@mountiny @allroundexperts Able to reproduce this issue in production Recording.860.mp4 |
I think this is the same as this one |
Job added to Upwork: https://www.upwork.com/jobs/~021869733355861718010 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @ntdiary ( |
Based on QA this is reliably repro on new accounts https://expensify.slack.com/archives/C049HHMV9SM/p1734614105597379?thread_ts=1734531561.566439&cid=C049HHMV9SM |
Marking this as CRITICAL quality issue as it makes the UX really bad |
@mountiny, If this is urgent, please feel free to reassign another c+. :) |
Thanks! SWM will have a look in the mean time 🙌 |
Someone from SWM will take a look at this shortly cc @BartoszGrajdek @Skalakid |
I can take this as I have context on it already! |
Cannot reproduce this both on new.expensify.com and staging.new.expensify.com, both on Chrome and Safari on macOS. |
PR is ready cc: @allroundexperts I'll open the upstream PR later and link it on the App PR. Done |
Hey @bernhardoj cross posting @allroundexperts comment from the PR -- can you give an ETA on when you'll be able to address this? This is a critical bug, so we are trying to get it unblocked ASAP:
|
Hey @bernhardoj bump on the above ^^ |
Sorry, I was OOO, addressed the comment. |
Just noting a second PR was opened and is under review: #55285 |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.86-3 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 2025-01-24. 🎊 For reference, here are some details about the assignees on this issue:
|
@allroundexperts @stephanieelliott @allroundexperts The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button] |
Payment Summary
BugZero Checklist (@stephanieelliott)
|
@allroundexperts, @stephanieelliott, @mountiny, @bernhardoj Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.89-8 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 2025-02-04. 🎊 For reference, here are some details about the assignees on this issue:
|
@allroundexperts @stephanieelliott @allroundexperts The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button] |
Payment Summary
BugZero Checklist (@stephanieelliott)
|
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number:
Reproducible in staging?: Needs Reproduction
Reproducible in production?: Needs Reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
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: @mountiny @quinthar
Slack conversation (hyperlinked to channel name): expensify_bugs
Action Performed:
Expected Result:
User able to type where the cursor
Actual Result:
The cursor jumps to the end of the text after a first character
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Screen.Recording.2024-12-18.at.15.15.47.mov
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @stephanieelliottThe text was updated successfully, but these errors were encountered: