fix(scrollBehavior): trigger scroll behavior if same route with hash (v…

…uejs#3592)

Co-authored-by: Ryan Klarhölter <[email protected]>

src/history/base.js

@@ -20,6 +20,7 @@ import {
   isNavigationFailure,
   NavigationFailureType
 } from '../util/errors'
+import { handleScroll } from '../util/scroll'
 
 export class History {
   router: Router
@@ -163,6 +164,9 @@ export class History {
       route.matched[lastRouteIndex] === current.matched[lastCurrentIndex]
     ) {
       this.ensureURL()
+      if (route.hash) {
+        handleScroll(this.router, current, route, false)
+      }
       return abort(createNavigationDuplicatedError(current, route))
     }
 

test/e2e/specs/scroll-behavior.js

@@ -94,14 +94,19 @@ module.exports = {
         'scroll to top on new entry'
       )
 
-      .click('li:nth-child(4) a')
-      .assert.evaluate(
-        function () {
-          return document.getElementById('anchor').getBoundingClientRect().top < 1
-        },
-        null,
-        'scroll to anchor'
-      )
+      .perform(() => {
+        for (let i = 0; i < 2; i++) {
+          browser
+            .click('li:nth-child(4) a')
+            .assert.evaluate(
+              function () {
+                return document.getElementById('anchor').getBoundingClientRect().top < 1
+              },
+              null,
+              (i === 0) ? 'scroll to anchor' : 'scroll to same anchor again'
+            )
+        }
+      })
 
       .click('li:nth-child(5) a')
       .assert.evaluate(