js/shoelace
1
0
Fork 0
mirror of https://github.com/shoelace-style/shoelace.git synced 2026-01-12 02:59:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

remove unused code path (#1539)

Browse Source
This commit is contained in:
Konnor Rogers
2023-08-23 12:52:42 -04:00
committed by GitHub
parent ae010c333b
commit f30481e229
1 changed files with 1 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/internal/tabbable.ts
View File

@@ -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;