Fix shallow routing scroll

packages/next/next-server/lib/router/router.ts

@@ -1093,13 +1093,16 @@ export default class Router implements BaseRouter {
           !(routeInfo.Component as any).getInitialProps
       }
 
+      // shallow routing is only allowed for same page URL changes.
+      const isValidShallowRoute = options.shallow && this.route === route
       await this.set(
         route,
         pathname!,
         query,
         cleanedAs,
         routeInfo,
-        forcedScroll || (options.scroll ? { x: 0, y: 0 } : null)
+        forcedScroll ||
+          (isValidShallowRoute || !options.scroll ? null : { x: 0, y: 0 })
       ).catch((e) => {
         if (e.cancelled) error = error || e
         else throw e

test/integration/client-navigation/pages/nav/shallow-routing.js

@@ -35,7 +35,9 @@ export default class extends Component {
             Home
           </a>
         </Link>
-        <div id="counter">Counter: {this.getCurrentCounter()}</div>
+        <div id="counter" style={{ marginBottom: 4000 }}>
+          Counter: {this.getCurrentCounter()}
+        </div>
         <div id="get-initial-props-run-count">
           getInitialProps run count: {this.props.getInitialPropsRunCount}
         </div>

test/integration/client-navigation/test/index.test.js

@@ -854,6 +854,21 @@ describe('Client Navigation', () => {
 
       await browser.close()
     })
+
+    it('should keep the scroll position on shallow routing', async () => {
+      const browser = await webdriver(context.appPort, '/nav/shallow-routing')
+      await browser.eval(() =>
+        document.querySelector('#increase').scrollIntoView()
+      )
+      const scrollPosition = await browser.eval('window.pageYOffset')
+
+      const newScrollPosition = await browser
+        .elementByCss('#increase')
+        .click()
+        .eval('window.pageYOffset')
+
+      expect(newScrollPosition).toBe(scrollPosition)
+    })
   })
 
   describe('with URL objects', () => {