Add test case and apply fixes

vercel · Feb 6, 2022 · 0d574a3 · 0d574a3
1 parent 7554cad
commit 0d574a3
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 6 deletions.
diff --git a/packages/next/client/link.tsx b/packages/next/client/link.tsx
@@ -96,11 +96,6 @@ function linkClicked(
 
   e.preventDefault()
 
-  //  avoid scroll for urls with anchor refs
-  if (scroll == null && as.indexOf('#') >= 0) {
-    scroll = false
-  }
-
   // replace state instead of push if prop is present
   router[replace ? 'replace' : 'push'](href, as, {
     shallow,

diff --git a/packages/next/shared/lib/router/router.ts b/packages/next/shared/lib/router/router.ts
@@ -1045,7 +1045,10 @@ export default class Router implements BaseRouter {
       this.asPath = cleanedAs
       Router.events.emit('hashChangeStart', as, routeProps)
       // TODO: do we need the resolved href when only a hash change?
-      this.changeState(method, url, as, options)
+      this.changeState(method, url, as, {
+        ...options,
+        scroll: false,
+      })
       if (scroll) {
         this.scrollToHash(cleanedAs)
       }

diff --git a/test/integration/client-navigation/pages/nav/hash-changes.js b/test/integration/client-navigation/pages/nav/hash-changes.js
@@ -27,6 +27,11 @@ const HashChanges = ({ count }) => {
       <Link href="#name-item-400">
         <a id="scroll-to-name-item-400">Go to name item 400</a>
       </Link>
+      <Link href="#name-item-400" scroll={false}>
+        <a id="scroll-to-name-item-400-no-scroll">
+          Go to name item 400 (no scroll)
+        </a>
+      </Link>
       <p>COUNT: {count}</p>
       {Array.from({ length: 500 }, (x, i) => i + 1).map((i) => {
         return (

diff --git a/test/integration/client-navigation/test/index.test.js b/test/integration/client-navigation/test/index.test.js
@@ -618,6 +618,17 @@ describe('Client Navigation', () => {
         }
       })
 
+      it('should not scroll to hash when scroll={false} is set', async () => {
+        const browser = await webdriver(context.appPort, '/nav/hash-changes')
+        const curScroll = await browser.eval(
+          'document.documentElement.scrollTop'
+        )
+        await browser.elementByCss('#scroll-to-name-item-400-no-scroll').click()
+        expect(curScroll).toBe(
+          await browser.eval('document.documentElement.scrollTop')
+        )
+      })
+
       it('should scroll to the specified position on the same page with a name property', async () => {
         let browser
         try {