Skip to content

fix init frontend TOC on DOMContentLoaded - #26

Merged
rsanchez-beapi merged 2 commits into
developfrom
fix/toc-init-domcontentloaded
Aug 28, 2026
Merged

rsanchez-beapi merged 2 commits into
developfrom
fix/toc-init-domcontentloaded

Conversation

@rsanchez-beapi

Copy link
Copy Markdown
Contributor

Context

Cette PR accélère l’initialisation front du bloc table des matières : le surlignage sticky des titres ne doit plus attendre la fin du chargement de toutes les ressources de la page.

Le TOC sticky devient actif plus tôt, sans changer le markup ni le comportement d’observation des titres.

Changement principaux

  • Remplacement de l’écoute window load par DOMContentLoaded dans src/view.js.
  • Ajout d’une garde document.readyState : si le document est déjà interactive / complete, l’init part tout de suite.

Note : l’init ne dépend que du DOM (liens TOC + IDs de titres). Attendre load n’apportait pas de garantie utile ici.

Critère d'acceptation

  • Sur une page avec TOC sticky + images lourdes, le titre actif se met à jour avant la fin du chargement des images.
  • Les liens TOC et getElementById ciblent toujours les bons titres.
  • Rechargement tardif / cache : le TOC s’initialise encore (pas de page « morte » si DOMContentLoaded est déjà passé).
  • npm run build + lint JS OK.

@rsanchez-beapi
rsanchez-beapi merged commit 44d8c71 into develop Aug 28, 2026
1 check passed
@rsanchez-beapi
rsanchez-beapi deleted the fix/toc-init-domcontentloaded branch August 28, 2026 12:12
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.

2 participants