Best would be using some jquery library for this: http://demos.flesler.com/jquery/scrollTo/

CommentFileSizeAuthor
#5 ajax_comment-2315677-fix_the_scrolling_to_new_post.patch829 bytesformatC'vt
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

formatC'vt’s picture

formatC'vt CreditAttribution: formatC'vt commented

what's wrong with jquery.animate?

muschpusch’s picture

muschpusch CreditAttribution: muschpusch commented

Nothing but it's pretty light weight and will correct margins etc for us + jumping to selectors will be easy.

https://github.com/flesler/jquery.scrollTo/blob/master/jquery.scrollTo.js

The scrolling is broken at the moment. If you wanna give it a try without any external dependencies please go ahead :)

formatC'vt’s picture

formatC'vt CreditAttribution: formatC'vt commented
Assigned: muschpusch » formatC'vt

ok, i will do it (today or tomorrow)

muschpusch’s picture

muschpusch CreditAttribution: muschpusch commented

nice!

formatC'vt’s picture

formatC'vt CreditAttribution: formatC'vt commented
Status: Active » Needs review
FileSize
ajax_comment-2315677-fix_the_scrolling_to_new_post.patch829 bytes

i work too hard and finally fix this strangest "javascript" bug, look at the patch =)

muschpusch’s picture

muschpusch CreditAttribution: muschpusch commented

Thanks I will try it tomorrow! It indeed looks complicated ;)

  • muschpusch committed de8d0a5 on 7.x-1.x authored by formatC'vt 
    Issue #2315677 by formatC'vt | muschpusch: Fixed the scrolling to form...
muschpusch’s picture

muschpusch CreditAttribution: muschpusch commented
Status: Needs review » Fixed

Thanks! Works great! Committed to dev

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.