Skip to content
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

[AWS] Remove delay / improve smoothness of scroll-to-top transition #5243

Closed
leecalcote opened this issue Dec 28, 2023 · 15 comments
Closed

[AWS] Remove delay / improve smoothness of scroll-to-top transition #5243

leecalcote opened this issue Dec 28, 2023 · 15 comments

Comments

@leecalcote
Copy link
Member

          Please be aware of the odd and delayed affect on reverse (on scroll back to the top).
Screen.Recording.2023-12-28.at.10.02.55.AM.mov

Originally posted by @leecalcote in #5225 (comment)

@Ashparshp
Copy link
Contributor

Hi @leecalcote,

I've reviewed the issue #5243 regarding the scroll-to-top transition, and it looks good to me. If there are any further details or changes you'd like, please let me know.

@leecalcote
Copy link
Member Author

@Ashparshp, thanks.
Given that the issue is imperceptible to your eyes, no, there’s no ask of you here, considering this requires finesse just beyond your reach.

@iArchitSharma
Copy link
Contributor

iArchitSharma commented Jan 1, 2024

@Ashparshp Issue occurs when the height is 1200 approx and above

@iArchitSharma
Copy link
Contributor

As discussed on Slack, I am assigning it to you

@Ashparshp
Copy link
Contributor

Ashparshp commented Jan 2, 2024

@leecalcote @iArchitSharma

Hi there! I was addressing a scrolling issue and made some adjustments in the code. Specifically, I changed the constant value from 16 to 8 in the formula const translateY = scrollPosition / 8;. This modification seems to have significantly improved the smoothness and reduced lag.

Video 👇
03.01.2024 00_42.webm
Hi there! I was addressing a scrolling issue and made some adjustments in the code. Specifically, I changed the constant value from 16 to 8 in the formula const translateY = scrollPosition / 8;. This modification seems to have significantly improved the smoothness and reduced lag.

@narasimha-1511
Copy link
Member

i want to work on this issue?

@Ashparshp
Copy link
Contributor

@narasimha-1511 Do you have access to Safari?

@narasimha-1511
Copy link
Member

narasimha-1511 commented Jan 20, 2024

@Ashparshp i did not understand your question

@Ashparshp
Copy link
Contributor

@narasimha-1511, the problem is specific to Safari, as discussed earlier. Can you confirm if you have access to Safari browser to check and work on this issue?

@narasimha-1511
Copy link
Member

yes i have access to safari web browser 😄

@Ashparshp
Copy link
Contributor

Great to hear, @narasimha-1511! 😄 @saurabh100ni Could you please take a look at this issue? Also, could you kindly ask @saurabh100ni to assign it as well? Thank you!

@saurabh100ni
Copy link
Contributor

Good to hear that. Thanks for the heads-up @Ashparshp. @narasimha-1511 , you up for the issue?

@narasimha-1511
Copy link
Member

@saurabh100ni can you please assign me the issue

@saurabh100ni
Copy link
Contributor

This issue has been open for some time with no recent activity, unassigning to open it up for new contributors to give it a go.

@leecalcote
Copy link
Member Author

Thanks for this @saurabh100ni. I suggest we take one step further and toss in the towel. I don’t think anyone’s gonna go get this done.

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

Successfully merging a pull request may close this issue.

5 participants