-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[$1000] IOS - IOUAmount - When entering/deleting numbers quickly, the cursor doubles #15856
Comments
Triggered auto assignment to @trjExpensify ( |
Bug0 Triage Checklist (Main S/O)
|
It's reproducible, but there seems to be some history on it here: #11146. We seem to have closed that issue due to a freeze on keyboard navigation, but I don't actually think it falls under that umbrella. Let's open this up. I understand the root cause might be upstream so that's the proposal we're probably looking for. Is that right? CC: @fedirjh @hannojg @thienlnam @Santhosh-Sellavel |
Looks like something related to As a reminder, please make sure that all proposals are not workarounds and that any and all attempt to fix the issue holistically have been made before proceeding with a solution. Proposals to change our Feel free to drop a note in #expensify-open-source with any questions. |
Job added to Upwork: https://www.upwork.com/jobs/~014612000725948717 |
Current assignee @trjExpensify is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak ( |
Triggered auto assignment to @roryabraham ( |
I agree with @trjExpensify that this looks like a valid bug report. Let's get some proposals here |
Hello, I'm Nicolay from Callstack and I'm interested in taking and analysing this issue to work on a fix for it. |
📣 @narefyev91 You have been assigned to this job by @mountiny! |
Is this issue open for proposals? |
📣 @huzaifa-99! 📣 Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Format:
|
This comment was marked as off-topic.
This comment was marked as off-topic.
@Christinadobrzyn, thanks! |
Still on hold for Fabric. |
Down to one major blocker on Fabric. |
Latest on Fabric: #8503 (comment) |
Still on HOLD, but we're getting closer to merging new arch |
Still held, latest on Fabric here. |
Update: according to #8503 (comment)
|
What's the latest status on switching to Fabric? I think this follow-up issue can go in #wave-collect as polish. |
Fabric is live, taking this off HOLD |
Alright, what are the next steps and who's doing them? Thanks! |
@trjExpensify, I cannot reproduce this bug anymore, the cursor looks normal Screen.Recording.2024-05-05.at.22.39.04.movThere's still something weird going on with the first symbol when entering numbers quickly ( |
On a real iOS device on the latest TF ( |
@trjExpensify, checked on a physical device, both bugs are not present |
Great, so we can close this then! |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Issue found when executing PR #15710
Action Performed:
Expected Result:
A second cursor doesn't "flash" after the first digit
Actual Result:
A second cursor appears after the first digit on every keyboard press
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.2.82.3
Reproducible in staging?: Yes
Reproducible in production?: Yes
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
Bug5971828_15710_iOS.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: