From afb5c0c001e1d955026b2562449773de88a6b754 Mon Sep 17 00:00:00 2001
From: Divyansh Singh <40380293+brc-dd@users.noreply.github.com>
Date: Sun, 14 Sep 2025 16:30:34 +0530
Subject: [PATCH] add comments

---
 src/client/app/router.ts | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/client/app/router.ts b/src/client/app/router.ts
index 577fc0f1..de7fe261 100644
--- a/src/client/app/router.ts
+++ b/src/client/app/router.ts
@@ -287,8 +287,12 @@ export function scrollTo(hash: string, smooth = false, scrollPosition = 0) {
 
     // focus the target element for better accessibility
     target.focus({ preventScroll: true })
+
+    // return if focus worked
     if (document.activeElement === target) return
 
+    // element has tabindex already, likely not focusable
+    // because of some other reason, bail out
     if (target.hasAttribute('tabindex')) return
 
     const restoreTabindex = () => {
@@ -296,10 +300,14 @@ export function scrollTo(hash: string, smooth = false, scrollPosition = 0) {
       target.removeEventListener('blur', restoreTabindex)
     }
 
+    // temporarily make the target element focusable
     target.setAttribute('tabindex', '-1')
     target.addEventListener('blur', restoreTabindex)
 
+    // try to focus again
     target.focus({ preventScroll: true })
+
+    // remove tabindex and event listener if focus still not worked
     if (document.activeElement !== target) restoreTabindex()
   }