fix: add tabIndex to VariantItem for accessibility

jpg619 · jpg619 · commit 591d55bf70ff · 2025-10-01T16:22:17.000-04:00
diff --git a/src/components/variant-select.js b/src/components/variant-select.js
@@ -16,7 +16,7 @@ const StyledOverlay = styled(ActionMenu.Overlay)`
 
 const VariantItem = ({title, shortName, url, active}) => {
   return (
-    <ActionList.Item state={{scrollUpdate: false}} id={shortName} active={active}>
+    <ActionList.Item state={{scrollUpdate: false}} id={shortName} active={active} tabIndex={null}>
       <LinkNoUnderline to={url}>{title}</LinkNoUnderline>
     </ActionList.Item>
   )

Original file line number	Diff line number	Diff line change
@@ -16,7 +16,7 @@ const StyledOverlay = styled(ActionMenu.Overlay)`
`16`	`16`
`17`	`17`	`const VariantItem = ({title, shortName, url, active}) => {`
`18`	`18`	`return (`
`19`		`- <ActionList.Item state={{scrollUpdate: false}} id={shortName} active={active}>`
	`19`	`+ <ActionList.Item state={{scrollUpdate: false}} id={shortName} active={active} tabIndex={null}>`
`20`	`20`	`<LinkNoUnderline to={url}>{title}</LinkNoUnderline>`
`21`	`21`	`</ActionList.Item>`
`22`	`22`	`)`