Skip to content

Conversation

@Han5991
Copy link
Contributor

@Han5991 Han5991 commented Nov 10, 2025

Fixes: #60667

  • Copy the documentation.md output to index.html/.json so /docs/latest/api/ immediately shows the
    introduction instead of a duplicated TOC
  • Harden tools/doc/allhtml.mjs and tools/doc/alljson.mjs against the new index structure by skipping
    anchor tags without href, replacing the TOC picker markup, and keeping anchor validation intact

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/web-infra

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. labels Nov 10, 2025
@codecov
Copy link

codecov bot commented Nov 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.01%. Comparing base (33f982e) to head (9cc3584).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #60673      +/-   ##
==========================================
- Coverage   88.53%   88.01%   -0.53%     
==========================================
  Files         703      703              
  Lines      208077   208077              
  Branches    40078    39995      -83     
==========================================
- Hits       184224   183131    -1093     
- Misses      15888    16923    +1035     
- Partials     7965     8023      +58     

see 105 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aduh95
Copy link
Contributor

aduh95 commented Nov 11, 2025

I'm not sure this is desirable, it current behavior is a much better UX for mobile users IMO

@Han5991
Copy link
Contributor Author

Han5991 commented Nov 11, 2025

I'm not sure this is desirable, it current behavior is a much better UX for mobile users IMO

I’d stick with the current layout. The duplicate TOC isn’t great on desktop, but on phones/tablets that list
is the only quick way to jump into the API docs. Without it, mobile users hit /api/ and have to open the picker
before they can navigate, which is a step backward

@aduh95
Copy link
Contributor

aduh95 commented Nov 11, 2025

COuldn't have said it better myself, I guess we can close this then

@aduh95 aduh95 closed this Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Render "About this documentation" as the landing page for the documentation section

3 participants