fix(a11y): nav-link anchor set focus on target

italia · Sep 25, 2023 · 51d69cf · 51d69cf
1 parent 2aa1234
commit 51d69cf
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/src/js/plugins/navscroll.js b/src/js/plugins/navscroll.js
@@ -93,8 +93,7 @@ class NavScroll extends BaseComponent {
     }
 
     SelectorEngine.find(SELECTOR_LINK_CLICKABLE, this._element).forEach((link) => {
-      link.addEventListener('click', (event) => {
-        event.preventDefault()
+      link.addEventListener('click', () => {
         const scrollHash = () => this._scrollToHash(link.hash)
         if (this._isCollapseOpened) {
           this._callbackQueue.push(scrollHash)
@@ -245,11 +244,9 @@ class NavScroll extends BaseComponent {
  * ------------------------------------------------------------------------
  */
 
-onDocumentScroll(() => {
-  const navs = SelectorEngine.find(SELECTOR_NAVSCROLL)
-  navs.map((nav) => {
-    NavScroll.getOrCreateInstance(nav)
-  })
+const navs = SelectorEngine.find(SELECTOR_NAVSCROLL)
+navs.map((nav) => {
+  NavScroll.getOrCreateInstance(nav)
 })
 
 export default NavScroll