Skip to content

🐛 Keyboard Navigation & Accessibility Issues in Interactive Components #657

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
AAdIprog wants to merge 2 commits into
base: main
Choose a base branch
from
Open

🐛 Keyboard Navigation & Accessibility Issues in Interactive Components #657

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a504dbf
fix(a11y): add keyboard navigation support for UseCases and VersionSe…
AAdIprog Jan 14, 2026
871f64f
trigger ci
AAdIprog Jan 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
87 changes: 60 additions & 27 deletions src/components/docs/VersionSelector.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,34 @@ export function VersionSelector({ className = '', isMobile = false }: VersionSel
return () => document.removeEventListener('keydown', handleEscape);
}, []);

// Keyboard navigation
const [focusedIndex, setFocusedIndex] = useState(-1);

// Reset focus when closed
useEffect(() => {
if (!isOpen) setFocusedIndex(-1);
}, [isOpen]);

useEffect(() => {
if (!isOpen) return;

function handleKeyDown(event: KeyboardEvent) {
if (event.key === 'ArrowDown') {
event.preventDefault();
setFocusedIndex(i => (i + 1) % versions.length);
} else if (event.key === 'ArrowUp') {
event.preventDefault();
setFocusedIndex(i => (i - 1 + versions.length) % versions.length);
} else if (event.key === 'Enter' && focusedIndex >= 0) {
event.preventDefault();
handleVersionChange(versions[focusedIndex].key);
}
}
Comment on lines +72 to +83
Copy link

Copilot AI Jan 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When the dropdown first opens and focusedIndex is -1, pressing ArrowDown or ArrowUp will set focusedIndex to 0 or the last index respectively, but there's no initial focused state when the dropdown opens. Consider initializing focusedIndex to 0 (or the current version's index) when the dropdown opens so users immediately have a focused option. This provides better keyboard navigation UX and matches standard dropdown behavior.

Copilot uses AI. Check for mistakes.

document.addEventListener('keydown', handleKeyDown);
return () => document.removeEventListener('keydown', handleKeyDown);
}, [isOpen, versions.length, focusedIndex]);
Copy link

Copilot AI Jan 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The useEffect dependency array includes focusedIndex but the effect uses handleVersionChange which is defined outside the effect. This creates a stale closure issue where handleVersionChange may reference outdated values. Additionally, handleVersionChange itself is not memoized and depends on pathname and projectId. Add handleVersionChange to the dependency array or move it inside the useEffect, or wrap handleVersionChange in useCallback with proper dependencies.

Copilot uses AI. Check for mistakes.

const handleVersionChange = (versionKey: string) => {
setIsOpen(false);

Expand All @@ -74,11 +102,10 @@ export function VersionSelector({ className = '', isMobile = false }: VersionSel
<div className={className}>
<button
onClick={() => setIsOpen(!isOpen)}
className={`flex items-center justify-between w-full px-3 py-2 text-sm rounded-md transition-colors ${
isDark
? 'text-gray-300 hover:bg-neutral-800'
: 'text-gray-700 hover:bg-gray-100'
}`}
className={`flex items-center justify-between w-full px-3 py-2 text-sm rounded-md transition-colors ${isDark
? 'text-gray-300 hover:bg-neutral-800'
: 'text-gray-700 hover:bg-gray-100'
}`}
>
<span className="flex items-center">
<svg className="w-4 h-4 mr-3" fill="none" stroke="currentColor" viewBox="0 0 24 24">
Expand All @@ -99,22 +126,26 @@ export function VersionSelector({ className = '', isMobile = false }: VersionSel

{isOpen && (
<div className="mt-1 ml-7 space-y-1">
{versions.map(({ key, label, externalUrl }) => {
{versions.map(({ key, label, externalUrl }, index) => {
const isCurrentVersion = key === 'latest';
const isExternal = !!externalUrl;
const isFocused = index === focusedIndex;
return (
<button
key={key}
onClick={() => handleVersionChange(key)}
className={`block w-full text-left px-3 py-2 text-sm rounded-md transition-colors ${
isCurrentVersion
className={`block w-full text-left px-3 py-2 text-sm rounded-md transition-colors ${isCurrentVersion
? isDark
? 'bg-neutral-800 text-white font-medium'
: 'bg-gray-100 text-gray-900 font-medium'
: isFocused
? isDark
? 'bg-neutral-800 text-white font-medium'
: 'bg-gray-100 text-gray-900 font-medium'
? 'bg-neutral-800 text-gray-200'
: 'bg-gray-100 text-gray-900'
: isDark
? 'text-gray-400 hover:bg-neutral-800 hover:text-gray-200'
: 'text-gray-600 hover:bg-gray-50 hover:text-gray-900'
}`}
}`}
>
{label}
{isExternal && (
Expand All @@ -136,11 +167,10 @@ export function VersionSelector({ className = '', isMobile = false }: VersionSel
<div className={`relative ${className}`} ref={dropdownRef}>
<button
onClick={() => setIsOpen(!isOpen)}
className={`flex items-center gap-1 text-xs font-mono px-2 py-1.5 rounded-md transition-colors cursor-pointer ${
isDark
? 'text-gray-400 bg-neutral-800/50 hover:bg-neutral-700 hover:text-gray-200'
: 'text-gray-600 bg-gray-100 hover:bg-gray-200 hover:text-gray-900'
}`}
className={`flex items-center gap-1 text-xs font-mono px-2 py-1.5 rounded-md transition-colors cursor-pointer ${isDark
? 'text-gray-400 bg-neutral-800/50 hover:bg-neutral-700 hover:text-gray-200'
: 'text-gray-600 bg-gray-100 hover:bg-gray-200 hover:text-gray-900'
}`}
aria-haspopup="listbox"
aria-expanded={isOpen}
aria-label={`Select ${currentProject.name} documentation version`}
Expand All @@ -159,32 +189,35 @@ export function VersionSelector({ className = '', isMobile = false }: VersionSel

{isOpen && (
<div
className={`absolute top-full right-0 mt-1 w-44 max-h-80 overflow-y-auto rounded-md shadow-lg border z-50 ${
isDark
? 'bg-neutral-900 border-neutral-700'
: 'bg-white border-gray-200'
}`}
className={`absolute top-full right-0 mt-1 w-44 max-h-80 overflow-y-auto rounded-md shadow-lg border z-50 ${isDark
? 'bg-neutral-900 border-neutral-700'
: 'bg-white border-gray-200'
}`}
role="listbox"
aria-label={`${currentProject.name} documentation versions`}
>
<div className="py-1">
{versions.map(({ key, label, externalUrl }) => {
{versions.map(({ key, label, externalUrl }, index) => {
const isCurrentVersion = key === 'latest';
const isExternal = !!externalUrl;
const isFocused = index === focusedIndex;

return (
<button
key={key}
onClick={() => handleVersionChange(key)}
className={`flex items-center justify-between w-full text-left px-3 py-2 text-sm transition-colors ${
isCurrentVersion
className={`flex items-center justify-between w-full text-left px-3 py-2 text-sm transition-colors ${isCurrentVersion
? isDark
? 'bg-neutral-800 text-white font-medium'
: 'bg-gray-100 text-gray-900 font-medium'
: isFocused
? isDark
? 'bg-neutral-800 text-white font-medium'
: 'bg-gray-100 text-gray-900 font-medium'
? 'bg-neutral-800 text-gray-200'
: 'bg-gray-100 text-gray-900'
: isDark
? 'text-gray-300 hover:bg-neutral-800'
: 'text-gray-700 hover:bg-gray-50'
}`}
}`}
role="option"
aria-selected={isCurrentVersion}
>
Expand Down
12 changes: 11 additions & 1 deletion src/components/master-page/UseCasesSection.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -267,7 +267,7 @@ export default function UseCasesSection() {
style={{ perspective: "1000px" }}
>
<div
className="relative w-full h-full transition-transform duration-700 ease-in-out cursor-pointer"
className="relative w-full h-full transition-transform duration-700 ease-in-out cursor-pointer rounded-2xl focus:outline-none focus:ring-4 focus:ring-blue-500 focus:ring-opacity-50"
style={{
transformStyle: "preserve-3d",
transform: flipped[index]
Expand All @@ -280,6 +280,16 @@ export default function UseCasesSection() {
}}
onMouseEnter={() => handleFlip(index, true)}
onMouseLeave={() => handleFlip(index, false)}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();
handleFlip(index, !flipped[index]);
}
}}
role="button"
tabIndex={0}
aria-pressed={flipped[index]}
aria-label={`View details for ${useCase.title}`}
>
{/* Front Face */}
<div
Expand Down