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

Link anchor issue on Process page #53453

Closed
NoNameProvided opened this issue Jun 14, 2024 · 5 comments
Closed

Link anchor issue on Process page #53453

NoNameProvided opened this issue Jun 14, 2024 · 5 comments
Labels
doc Issues and PRs related to the documentations.

Comments

@NoNameProvided
Copy link

Affected URL(s)

https://nodejs.org/docs/latest-v22.x/api/process.html

Description of the problem

When opening the documentation page about Process and clicking the process.send(message[, sendHandle[, options]][, callback]) option in the menu the page scrolls to the bottom instead of the correct section.

After clicking around between the menu options the behavior seems to be inconsistent:

  • some menu options seem to work correctly (anchor is on top, just outside of viewport, I assume this is the expected behavior)
  • some menu options are scrolled to the "center" or 2/3 position of the viewport
  • some menu options don't seem to work at all

After a quick peek at the source, the anchor tag seems correct in the source file, so I am unsure why it works as is.

I have tested in Edge (Chromium), Chrome, and Firefox and observed the same behavior in all.

@NoNameProvided NoNameProvided added the doc Issues and PRs related to the documentations. label Jun 14, 2024
@VoltrexKeyva VoltrexKeyva added the process Issues and PRs related to the process subsystem. label Jun 14, 2024
@avivkeller
Copy link
Member

Thanks for opening an issue!

Is this only the process page, or is it scattered throughout the docs?

@Septh
Copy link

Septh commented Jun 16, 2024

It does happen with every page in the docs. Also, the smooth scrolling feature that was recently introduced is a real pain, if you don't mind my saying.

@avivkeller avivkeller removed the process Issues and PRs related to the process subsystem. label Jun 16, 2024
@avivkeller
Copy link
Member

The @nodejs/web-infra team is working a redesign, so I've pinged them to take a look.

@tniessen
Copy link
Member

This might be a duplicate of #47106 or perhaps even of #40099.

@ovflowd
Copy link
Member

ovflowd commented Jun 17, 2024

Duplicate of #40099

@ovflowd ovflowd marked this as a duplicate of #40099 Jun 17, 2024
@ovflowd ovflowd closed this as not planned Won't fix, can't repro, duplicate, stale Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Development

No branches or pull requests

6 participants