-
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
[$250] Android - Taxes - Cursor is placed before the value when reopening saved value #39081
Comments
Triggered auto assignment to @MitchExpensify ( |
@MitchExpensify FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
We think that this bug might be related to #wave-control |
This comment was marked as outdated.
This comment was marked as outdated.
Job added to Upwork: https://www.upwork.com/jobs/~01515a93e0ea4bae26 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @cubuspl42 ( |
Proposal Updated
|
Not reproducible on main in my case: Screen.Recording.2024-03-31.at.12.54.40.AM.mov |
Proposal Updated
|
Proposal Updated
|
Proposals ready for you @cubuspl42 |
@cubuspl42 below Profile and Members you should also see More Features. Enable Taxes under More Features. My collect workspace was created on old dot. I don't know if that makes a difference. |
This comment was marked as outdated.
This comment was marked as outdated.
@cubuspl42, I guess you are not the admin of the workspace. |
I thought "owner" and "admin" are the same. You can see that I'm the owner on the screenshots. |
I found a solution; see this Slack message. I'll see if I can reproduce this issue now. |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@lanitochka17 @MitchExpensify I can't reproduce, same as in this comment.. I tested on latest |
@Krishna2323 Just wanna confirm, are you going to update your proposal again with this comment #39081 (comment)? Btw, could you confirm if you can still reproduce this bug in the latest main branch? Thanks a lot. |
@hoangzinh, I wasn't able to reproduce this issue in other amount forms but this is still reproducible on Tax amount page. I don't think we want to make the changes in |
Not overdue, actively being worked on and the ball is in @hoangzinh 's court to consider @Krishna2323 's response above |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Friendly bump @hoangzinh |
Issue not reproducible during KI retests. (First week) |
@Krishna2323 can you find any upstream issue link mentioned to this bug? |
Friendly bump @Krishna2323 |
@hoangzinh, I couldn't find any upstream issue link. |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@lanitochka17 sorry for asking again, can you still reproduce this bug? |
Issue is still reproducible Screen_Recording_20240509_203927_New.Expensify.mp4 |
@MitchExpensify I'm unable to reproduce this bug on my end. To speed up the progress of this issue, Can I ask in C+ group if anyone can reproduce then they can pick up this issue, what do you think? |
Good idea @hoangzinh , shared here https://expensify.slack.com/archives/C02NK2DQWUX/p1715532975658179 |
I couldn't reproduce it at my end on physical Android device, prod 1.4.71-6. @lanitochka17 Could you share some details on your device and the app version? |
Was able to reproduce this on my Android emulator. @MitchExpensify Can you add the |
Issue not reproducible during KI retests. (Second week) |
@mvtglobally I am able to reproduce it on my emulator. Screen.Recording.2024-05-14.at.1.07.11.AM.mov |
Given this is such minor issue, isolated to Android, and we're having trouble replicating it I don't think this is worth our time |
@MitchExpensify, what do you think about the other bug, the amount input is not cleared when we go back without saving it. I can still reproduce the cursor bug easily on my physical device and emulator. edited_amount_not_cleared_tax_page.mp4 |
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.57-2
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Issue found when executing PR #38733
Action Performed:
Precondition:
Expected Result:
Cursor is placed behind the value
Actual Result:
Cursor is placed in front of the value. Sometimes it appears behind the value
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6428815_1711552320921.Screen_Recording_20240327_230730_New_Expensify.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: