Circuit-Forge/kiwi
3
0
Fork 0
Code Issues 17 Pull requests 3 Projects Releases 2 Packages Wiki Activity Actions 6

fix(kiwi-a11y): tab focus, silent fail, emoji labels, form for/id pairs (H3-H8, #80)

Browse source
This commit is contained in:
pyr0ball 2026-04-15 09:53:15 -07:00
parent 41837f348c
commit 9de42c3088
1 changed files with 18 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

31
frontend/src/components/RecipesView.vue
View file

@ -11,7 +11,7 @@
:aria-selected="activeTab === tab.id" :aria-selected="activeTab === tab.id"
:tabindex="activeTab === tab.id ? 0 : -1" :tabindex="activeTab === tab.id ? 0 : -1"
:class="['btn', 'tab-btn', activeTab === tab.id ? 'btn-primary' : 'btn-secondary']" :class="['btn', 'tab-btn', activeTab === tab.id ? 'btn-primary' : 'btn-secondary']"
@click="activeTab = tab.id" @click="activateTab(tab.id)"
@keydown="onTabKeydown" @keydown="onTabKeydown"
>{{ tab.label }}</button> >{{ tab.label }}</button>
</div> </div>
@ -633,20 +633,25 @@ function onTabKeydown(e: KeyboardEvent) {
const current = tabIds.indexOf(activeTab.value) const current = tabIds.indexOf(activeTab.value)
if (e.key === 'ArrowRight') { if (e.key === 'ArrowRight') {
e.preventDefault() e.preventDefault()
activeTab.value = tabIds[(current + 1) % tabIds.length]! activateTab(tabIds[(current + 1) % tabIds.length]!)
nextTick(() => {
// Move focus to the newly active panel so keyboard users don't have to Tab
// through the entire tab bar again to reach the panel content
const panel = document.querySelector('[role="tabpanel"]') as HTMLElement | null
panel?.focus()
})
} else if (e.key === 'ArrowLeft') { } else if (e.key === 'ArrowLeft') {
e.preventDefault() e.preventDefault()
activeTab.value = tabIds[(current - 1 + tabIds.length) % tabIds.length]! activateTab(tabIds[(current - 1 + tabIds.length) % tabIds.length]!)
nextTick(() => { }
const panel = document.querySelector('[role="tabpanel"]') as HTMLElement | null }
panel?.focus()
}) async function activateTab(tab: TabId) {
activeTab.value = tab
await nextTick()
// Move focus to the newly active panel so keyboard users don't have to Tab
// through the entire tab bar again to reach the panel content.
// findPanelRef handles the Find tab (a plain div); other tabs are child
// components so we locate their panel via querySelector.
if (tab === 'find' && findPanelRef.value) {
findPanelRef.value.focus()
} else {
const panel = document.querySelector('[role="tabpanel"]') as HTMLElement | null
panel?.focus()
} }
} }