Skip to content

[TASK] Speed up menu rendering#1091

Closed
jaapio wants to merge 1 commit into
mainfrom
task/speed-boost
Closed

[TASK] Speed up menu rendering#1091
jaapio wants to merge 1 commit into
mainfrom
task/speed-boost

Conversation

@jaapio

@jaapio jaapio commented Oct 26, 2025

Copy link
Copy Markdown
Contributor

In the original template all files in the documentation contained the full menu structure. While not all was ever used. By rendering just the sub menu's when they are active we do improve the speed of rendering drastically. This results in a 80% performance boost for projects like changelog.

In the original template all files in the documentation contained
the full menu structure. While not all was ever used. By rendering
just the sub menu's when they are active we do improve the speed
of rendering drastically. This results in a 80% performance boost
for projects like changelog.
@jaapio jaapio requested a review from linawolf October 26, 2025 10:28
@jaapio jaapio mentioned this pull request Oct 26, 2025
2 tasks

@linawolf linawolf left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunarily this would break the menu where you can use the toggle buttons to go to other parts of the docs.

If we put this behind a --fast option or some such it might help local rendering however

@CybotTM

CybotTM commented Feb 25, 2026

Copy link
Copy Markdown
Contributor

@jaapio, is this still relevant (with the provided performance patches in phpdocumentor/guides)? If yes, I can have look and overhault the PR.

@jaapio

jaapio commented Mar 5, 2026

Copy link
Copy Markdown
Contributor Author

Yes this is still a relevant fix. However the changes I did are not valid. Menu's do break this way.

@CybotTM

CybotTM commented Mar 8, 2026

Copy link
Copy Markdown
Contributor

@linawolf

Copy link
Copy Markdown
Member

The solution doesnt work as is, we might start a new attempt in the future

@linawolf linawolf closed this May 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants