Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

app-directory: <Link /> in full height header navigation moves down to content althrough scroll={false} is set #50398

Open
1 task done
patshologram opened this issue May 26, 2023 · 1 comment
Labels
bug Issue was opened via the bug report template.

Comments

@patshologram
Copy link

Verify canary release

  • I verified that the issue exists in the latest Next.js canary release

Provide environment information

Operating System:
      Platform: darwin
      Arch: arm64
      Version: Darwin Kernel Version 22.5.0: Mon Apr 24 20:52:24 PDT 2023; root:xnu-8796.121.2~5/RELEASE_ARM64_T6000
    Binaries:
      Node: 18.15.0
      npm: 9.6.6
      Yarn: 1.22.19
      pnpm: N/A
    Relevant packages:
      next: 13.4.5-canary.0
      eslint-config-next: N/A
      react: 18.2.0
      react-dom: 18.2.0
      typescript: 4.9.5

Which area(s) of Next.js are affected? (leave empty if unsure)

No response

Link to the code that reproduces this issue or a replay of the bug

https://github.com/patshologram/link-scroll

To Reproduce

  1. check out https://github.com/patshologram/link-scroll
  2. run yarn
  3. run yarn dev
  4. open http://localhost:3000
  5. click the links on top and see how the page is scrolled down to the bottom/content

Describe the Bug

We implemented an animated mobile menu which spans across the whole height of the device and pushes the content down. We discovered that because of the animation the navigation always lands somewhere between the header and the content when navigating between sites.

Expected Behavior

In my understanding, there should not be any scrolling if scroll={false} is set, no matter how tall the navigation component is.

Which browser are you using? (if relevant)

No response

How are you deploying your application? (if relevant)

No response

@patshologram patshologram added the bug Issue was opened via the bug report template. label May 26, 2023
@lsagetlethias
Copy link

Maybe linked to #50105

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue was opened via the bug report template.
Projects
None yet
Development

No branches or pull requests

2 participants