[SelectField] Do not allow wrapping button container to be keyboard…

… focusable
techniq · Jul 16, 2024 · 95447db · 95447db
1 parent e19e7dd
commit 95447db
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.changeset/serious-boxes-rush.md b/.changeset/serious-boxes-rush.md
@@ -0,0 +1,5 @@
+---
+'svelte-ux': patch
+---
+
+[SelectField] Do not allow wrapping `button` container to be keyboard focusable
diff --git a/packages/svelte-ux/src/lib/components/SelectField.svelte b/packages/svelte-ux/src/lib/components/SelectField.svelte
@@ -412,7 +412,6 @@
   }
 </script>
 
-<!-- svelte-ignore a11y-click-events-have-key-events -->
 <button
   type="button"
   aria-haspopup={!inlineOptions ? 'listbox' : undefined}
@@ -424,6 +423,7 @@
   )}
   bind:this={selectFieldEl}
   on:click={onClick}
+  tabindex="-1"
 >
   <TextField
     {label}