Router scrollBehavior to.hash fail after #272

[meteorlxy]

I notice that because of #291. I find the sidebar link's default behavior fails.

Recent commit about that is #272, merged in 6b6d268

[meteorlxy]

vuepress/lib/default-theme/Layout.vue

Lines 179 to 181 in 6b6d268

	if (isActive && this.$route.hash !== anchor.hash) {
	this.$router.replace(anchor.hash)
	return

vuepress/lib/app/app.js

Lines 56 to 64 in 6b6d268

	scrollBehavior: (to, from, saved) => {
	if (saved) {
	return saved
	} else if (to.hash) {
	return false
	} else {
	return { x: 0, y: 0 }
	}
	}

$router.replace will trigger scrollBehavior, so he set the to.hash branch false, which disabled the default behavior of sidebar links.

Simply set it back to { selector: to.hash } may cause some scroll issue.

[ghost]

It would be nice to restore the { selector: to.hash } scroll behavior, but prevent it when we're making the $router.replace call while scrolling through the page. With the normal { selector: to.hash }, it will scroll to the anchor when we do the $router.replace while we're scrolling, which leads to some jerky page jumps (especially when scrolling up).

Is there a way to do this? I've looked into it, and I haven't found any obvious solutions.

[ulivz]

Fixed at 6081a3d.