remove unused code path (#1539)

shoelace-style · Aug 23, 2023 · f30481e · f30481e
1 parent ae010c3
commit f30481e
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/src/internal/tabbable.ts b/src/internal/tabbable.ts
@@ -69,7 +69,6 @@ export function getTabbableBoundary(root: HTMLElement | ShadowRoot) {
 }
 
 export function getTabbableElements(root: HTMLElement | ShadowRoot) {
-  const allElements: HTMLElement[] = [];
   const tabbableElements: HTMLElement[] = [];
 
   function walk(el: HTMLElement | ShadowRoot) {
@@ -79,10 +78,6 @@ export function getTabbableElements(root: HTMLElement | ShadowRoot) {
         return;
       }
 
-      if (!allElements.includes(el)) {
-        allElements.push(el);
-      }
-
       if (!tabbableElements.includes(el) && isTabbable(el)) {
         tabbableElements.push(el);
       }
@@ -116,7 +111,7 @@ export function getTabbableElements(root: HTMLElement | ShadowRoot) {
 
   // Is this worth having? Most sorts will always add increased overhead. And positive tabindexes shouldn't really be used.
   // So is it worth being right? Or fast?
-  // return allElements.filter(isTabbable).sort((a, b) => {
+  // return tabbableElements.filter(isTabbable).sort((a, b) => {
   //   // Make sure we sort by tabindex.
   //   const aTabindex = Number(a.getAttribute('tabindex')) || 0;
   //   const bTabindex = Number(b.getAttribute('tabindex')) || 0;