User-scrolling after the scroll-up [#1944146]

I seeing some weird behavior in Chrome, where after I scroll back to the top, there's a second or two where if I try to manually scroll back down, the jQuery animation seems to fight against me and keeps it at the top. This results in a jumpy/stuttery effect, where it won't let me scroll down again, until that time elapses. It seems like the animation really slows down at the end and takes its time in giving me back the control. In Firefox it is similar, but without the stuttery effect (Firefox just doesn't register manual scrolling until after the animation is completely finished).

It alleviates the problem if I increase the speed of the animation to 200 (instead of the current 1200). I actually made it a configurable setting in the admin page. I'd be happy to post those changes as a patch if desired.

To clarify, I don't think this is a bug with this module, just weirdness with jQuery, animate, and scrolling. However it seems it can be alleviated with a faster animation speed. Is anyone else experiencing this?

Comment	File	Size	Author
#5	back_to_top-animation_settings-1944146-5.patch	3.56 KB	brockfanning
#4	back_to_top-configurable_animation_speed-1944146-4.patch	1.89 KB	brockfanning

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Lux Delux CreditAttribution: Lux Delux commented 18 March 2013 at 17:19

Just noticed this today as well. Thanks for the tip about the animation speed. Changed it to 200 in the modules js and everything works fine

Comment #2

semei CreditAttribution: semei commented 11 May 2013 at 12:30

a patch would be great, this should definitely be commited. i experience the same issue.

Comment #3

semei CreditAttribution: semei commented 11 May 2013 at 12:32

same issue when i chance the speed of animation from 1200 to 200, it just fixes itself faster than before.

Comment #4

brockfanning CreditAttribution: brockfanning commented 17 May 2013 at 17:12

Version:	7.x-1.3	» 7.x-1.x-dev
Status:	Active	» Needs review

File	Size
back_to_top-configurable_animation_speed-1944146-4.patch	1.89 KB

Ok, here's what I had come up with. Let me know if anything doesn't work right.

Comment #5

brockfanning CreditAttribution: brockfanning commented 2 August 2013 at 16:35

Category:

task

» feature

File	Size
back_to_top-animation_settings-1944146-5.patch	3.56 KB

Here's a new patch that also makes the "easing" configurable. I found that while tweaking the speed helped with the problem, removing the easing altogether pretty much solved it for me.

Comment #6

mjcarter CreditAttribution: mjcarter commented 6 November 2013 at 11:27

Issue summary:	View changes
Status:	Needs review	» Reviewed & tested by the community

Works as described. Doesn't fix the root of the issue but the configuration options are a nice and useful addition and can mitigate or work around the issue.

Comment #7

acke CreditAttribution: acke commented 2 January 2014 at 16:43

I think I fixed the root of the problem with a bind / unbind. It's committed and pushed to HEAD/7.x-dev and going into 1.4.

Making the speed and animation type configurable is also a good idea. But maybe more or all easing type should be included (http://jqueryui.com/resources/demos/effect/easing.html)?