Next/previous shortcuts cause annoying page scroll

[sid-kap]

When I press [[ or ]], the page usually scrolls down a bit, even if the "Previous" or "Next" text was already in the viewport.

To reproduce: go to http://abstrusegoose.com/570, increase zoom to around 150% (large enough so that the page doesn't fit in a single frame, and the scrollbar appears), and then hit [[ or ]].

What might be causing this? How could I fix this?

[smblott-github]

@mrmr1993...

It looks like that's pretty old code here. Do we know that the scroll and focus are needed? It seems to still work without them.

[mrmr1993]

I assume the intention was to show which link we are following (especially useful when users report the wrong link is selected, so they can tell us which).

I vaguely remember that element.scrollIntoFocus doesn't always scroll or not as expected, but I'm not sure what should be done.