Scroll position preserving doesn't match browser behavior

[gaearon]

Browser only preserves scrolling position when you go Back or Forward.
If you open the same link twice, browser will scroll it to top.

However, react-router seems to always remember scroll position for every URL.
This means if you scrolled some page, it will forever open scrolled from now on.

I believe react-router should only attempt restoring scroll position if path changed due to popstate, and not due to pushstate.

[ryanflorence]

I think we already have an issue for this

[mjackson]

@gaearon I opened #252 to try and nail down the cases where we're scrolling and we shouldn't be, but so far that ticket only mentions clicking on links. Is that what you mean by "open the same link twice"? Or are you doing that via back/forward buttons?

[gaearon]

@mjackson

Click on a link.
Scroll down.
Go back.
Click on it again.

Expected: scrollY = 0, actual: scrollY is restored.

[gaearon]

Right, it's a dupe of #252. I didn't notice it :-(