fix for "scroll to page anchor"

[Wes1262]

...and starting an attempt to clean this file.

[morrisonlevi]

Thanks for looking into this. I am not sure about some of the details; the callbackCalled = true; looks shady. I'm also not completely sure that this will prevent the browser's default scroll in some cases. I'll try and find some time to play with this to see how it works and if I can it to break.

[Wes1262]

yes, because i don't know why $('html, body') was used. maybe for compatibility with some old browser. that causes the callback to be executed twice which is not what we want so for now i just prevented that to happen. clearly it's a temporary solution till i can investigate more (it's not different from previous implementation anyway)

also. i am 100% sure the approach for preventing default scrolling is correct

[Wes1262]

I've removed this because the hashchange event is not cancelable. It couldn't have ever worked