Skip to content

PoC: Lazy loaded navigation #1505

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Jul 2, 2025
Merged

PoC: Lazy loaded navigation #1505

merged 4 commits into from
Jul 2, 2025

Conversation

reakaleek
Copy link
Member

@reakaleek reakaleek commented Jul 2, 2025 •
edited
Loading

Context

In https://www.elastic.co/docs/reference/, we have a huge navigation with 7069 links that impacts initial loading time.

image

Changes

  • Only render 2 levels of the navigation initially (The root level and first level)
  • Create a partial html file called index.nav.html for every html file.
  • Load the partial navigation with htmx on load.

Notes

This feature is hidden behind the LAZY_LOAD_NAVIGATION feature flag and will be activated for our edge environment.

@reakaleek reakaleek requested a review from a team as a code owner July 2, 2025 13:49
@reakaleek reakaleek marked this pull request as draft July 2, 2025 13:50
@reakaleek reakaleek self-assigned this Jul 2, 2025
@reakaleek reakaleek marked this pull request as ready for review July 2, 2025 16:10
@reakaleek reakaleek merged commit 4e9dcc9 into main Jul 2, 2025
15 checks passed
@reakaleek reakaleek deleted the feature/lazy-loaded-navigation branch July 2, 2025 16:10
@reakaleek reakaleek mentioned this pull request Jul 7, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cotti cotti cotti approved these changes

Assignees

@reakaleek reakaleek

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@reakaleek @cotti