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

doc: avoid hiding by navigation bar in anchor jumping in 17.x #53663

Open
wants to merge 3 commits into
base: v17.x
Choose a base branch
from

Conversation

cloydlau
Copy link
Contributor

@cloydlau cloydlau commented Jul 1, 2024

See: #45131 (comment)

Merging into the v17.x branch should only affect the documentation for Node 17.

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/nodejs-website
@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jul 1, 2024
@richardlau
Copy link
Member

Node.js 17 is End-of-Life and will not get new versions.

@cloydlau
Copy link
Contributor Author

cloydlau commented Jul 4, 2024

It's sad but the original PR was proposed in 2022. I guess at that time, Node 17 hadn't reached EOL yet?

Two years later, the original PR was suddenly merged into the main branch (targeting the current 22.x), which caused unexpected issues.

@jwueller
Copy link
Contributor

jwueller commented Jul 10, 2024

Node.js 17 is End-of-Life and will not get new versions.

It is still valuable to at least have functional documentation for older versions, even if it's just for reference. Why is this tied to Node.js releases?

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.
6 participants