Skip to content

[$25] During updating the details in jobs tab, the data entered gets lost as soon as zoom in/out on the page(before it is saved) #5796

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

Closed
LieutenantRoger opened this issue Nov 12, 2021 · 23 comments
Assignees
Milestone

Comments

@LieutenantRoger
Copy link
Collaborator

  1. Update your jobs profile by entering data into all the fields as shown below,
    https://www.topcoder.com/settings/profile#work
    image

  2. Before saving the data, Then try zooming in/out by pressing by pressing 'ctrl' and scrolling through mouse scroller. All the data entered gets lost

image

@LieutenantRoger LieutenantRoger added Functional P2 Important (resolve within 3 days) Valid FE labels Nov 12, 2021
@LieutenantRoger LieutenantRoger changed the title During updating the details in jobs tab, the data entered gets lost as soon as zoom in/out on the page(before it is saved) [$25] During updating the details in jobs tab, the data entered gets lost as soon as zoom in/out on the page(before it is saved) Nov 15, 2021
@LieutenantRoger
Copy link
Collaborator Author

Challenge https://www.topcoder.com/challenges/b16c606e-8865-426f-b643-17018a4a860a has been created for this ticket.

This is an automated message for lieutenantroger via Topcoder X

@fikzzzy
Copy link
Contributor

fikzzzy commented Nov 16, 2021

@bug-bash-helper assign

@bug-bash-helper
Copy link

@fikzzzy ✅ you are now assigned to this issue and have 12 hours to complete it.

As soon as you are done, please, make a comment like below, including the link to the pull request:

@bug-bash-helper <link to PR> is ready for review

1 similar comment
@bug-bash-hunt-helper
Copy link

@fikzzzy ✅ you are now assigned to this issue and have 12 hours to complete it.

As soon as you are done, please, make a comment like below, including the link to the pull request:

@bug-bash-helper <link to PR> is ready for review

@fikzzzy
Copy link
Contributor

fikzzzy commented Nov 16, 2021

unassign. after ive tried, every time the browser zoom change, react will trigger its render function. So it is too difficult to check of render condition

@fikzzzy
Copy link
Contributor

fikzzzy commented Nov 16, 2021

@bug-bash-helper unassign

@bug-bash-helper
Copy link

@fikzzzy ✅ you have been unassigned from this issue.

Now you may pick up another issue which is open for pickup if you like to.

1 similar comment
@bug-bash-hunt-helper
Copy link

@fikzzzy ✅ you have been unassigned from this issue.

Now you may pick up another issue which is open for pickup if you like to.

@yoution
Copy link
Contributor

yoution commented Nov 17, 2021

@bug-bash-helper assign

@bug-bash-helper
Copy link

@yoution ✅ you are now assigned to this issue and have 12 hours to complete it.

As soon as you are done, please, make a comment like below, including the link to the pull request:

@bug-bash-helper <link to PR> is ready for review

1 similar comment
@bug-bash-hunt-helper
Copy link

@yoution ✅ you are now assigned to this issue and have 12 hours to complete it.

As soon as you are done, please, make a comment like below, including the link to the pull request:

@bug-bash-helper <link to PR> is ready for review

@LieutenantRoger
Copy link
Collaborator Author

Challenge https://www.topcoder.com/challenges/b16c606e-8865-426f-b643-17018a4a860a has been assigned to yoution.

This is an automated message for lieutenantroger via Topcoder X

@LieutenantRoger
Copy link
Collaborator Author

Challenge https://www.topcoder.com/challenges/b16c606e-8865-426f-b643-17018a4a860a yoution has been unassigned.

This is an automated message for lieutenantroger via Topcoder X

@codejamtc
Copy link
Collaborator

Fixed 🆗

bandicam.2021-11-18.10-50-55-515.mp4

@LieutenantRoger
Copy link
Collaborator Author

Challenge https://www.topcoder.com/challenges/b16c606e-8865-426f-b643-17018a4a860a has been assigned to yoution.

This is an automated message for lieutenantroger via Topcoder X

LieutenantRoger added a commit that referenced this issue Nov 19, 2021
* Enable Clear Date Input (#5809)

* Validate Non Valid Input Characters (#5810)

* ci:deploying

* fix: issue #5807 (#5811)

* fix: issue #5802 (#5812)

* fix: issue #5796 (#5813)

* fix: #5798 (#5814)

* fix:5798-2

* fix: test

* fix: issue #5800 (#5816)

* fix: issue #5786 (#5815)

* fix: issue #5802 (#5823)

* deploying ca-branch

* revert ci

Co-authored-by: M Fikri A <[email protected]>
Co-authored-by: yoution <[email protected]>
Co-authored-by: Shivam Kumar Singh <[email protected]>
@codejamtc codejamtc added this to the v1.14.3 milestone Nov 19, 2021
@SathyaJayabal SathyaJayabal added QA Fail QA verification on Dev has failed. Assignee to redo the fix. and removed QA Pass in Dev labels Nov 22, 2021
@SathyaJayabal
Copy link
Collaborator

Still see this issue when you zoom into 200% and the mobile view is displayed. @LieutenantRoger

zoom.mov

This was referenced Nov 22, 2021
@LieutenantRoger
Copy link
Collaborator Author

Still see this issue when you zoom into 200% and the mobile view is displayed. @LieutenantRoger

zoom.mov

@SathyaJayabal This is expected since switching from Web view into Mobile view will utilize a differnt component, hence the original component state will loss as expected.

I think let's accept this for now, it's a very rare scenario that a user will switch from webview into mobileview when updating the work info in his own profile page.

@LieutenantRoger LieutenantRoger added Ready for QA and removed QA Fail QA verification on Dev has failed. Assignee to redo the fix. labels Nov 22, 2021
@coderhacker
Copy link

This is passed on to production

5796.mp4

n

@LieutenantRoger
Copy link
Collaborator Author

Payment task has been updated: https://www.topcoder.com/challenges/b16c606e-8865-426f-b643-17018a4a860a
Payments Complete
Winner: yoution
Copilot: lieutenantroger
Challenge b16c606e-8865-426f-b643-17018a4a860a has been paid and closed.

This is an automated message for lieutenantroger via Topcoder X

@coderhacker coderhacker added QA Pass in PROD Passed verification on Production and removed tcx_Paid labels Nov 23, 2021
@LieutenantRoger
Copy link
Collaborator Author

@coderhacker please don't remove tcx_Paid label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants