From ddf5622b2f74b0e4772f1102a0a9a7d4794abd9a Mon Sep 17 00:00:00 2001 From: Darius Cepulis Date: Tue, 14 Apr 2026 09:50:53 -0500 Subject: [PATCH] fix(site): site accessibility rough edges (#1330) Co-authored-by: Claude Opus 4.6 (1M context) --- site/src/assets/logos/videojs-mono.svg | 1 + site/src/components/CopyButton.tsx | 27 ++++--- site/src/components/CopyMarkdownButton.tsx | 77 ++++++++++--------- site/src/components/DialNav.tsx | 2 + site/src/components/Footer.astro | 9 ++- site/src/components/FooterDocs.astro | 9 ++- site/src/components/NavBar/MobileNav.tsx | 4 +- site/src/components/NavBar/NavBar.astro | 6 +- site/src/components/NavBar/NavBarDocs.astro | 11 ++- site/src/components/Tabs.tsx | 11 +-- site/src/components/ThemeToggle.tsx | 1 + site/src/components/docs/DocsNavigation.astro | 5 +- site/src/components/docs/DocsSidebar.astro | 2 +- site/src/components/docs/SidebarItem.astro | 6 +- .../components/home/GetStartedBanner.astro | 1 + site/src/components/home/Header.astro | 19 ++--- site/src/components/home/Sponsors.astro | 25 ++++-- site/src/layouts/AuthResult.astro | 5 +- site/src/layouts/Base.astro | 6 ++ site/src/layouts/Blog.astro | 2 +- site/src/layouts/ErrorPage.astro | 2 +- site/src/layouts/Markdown.astro | 2 +- .../framework/[framework]/[...slug].astro | 1 + site/src/pages/index.astro | 18 ++++- site/src/pages/support.astro | 7 +- 25 files changed, 168 insertions(+), 91 deletions(-) diff --git a/site/src/assets/logos/videojs-mono.svg b/site/src/assets/logos/videojs-mono.svg index 6945d217..97dce3ba 100644 --- a/site/src/assets/logos/videojs-mono.svg +++ b/site/src/assets/logos/videojs-mono.svg @@ -1,4 +1,5 @@ + Video.js diff --git a/site/src/components/CopyButton.tsx b/site/src/components/CopyButton.tsx index d4505718..88a37948 100644 --- a/site/src/components/CopyButton.tsx +++ b/site/src/components/CopyButton.tsx @@ -55,16 +55,21 @@ export default function CopyButton({ children, copied, copyFrom, className, styl }; return ( - + <> + + + {isCopied ? 'Copied' : ''} + + ); } diff --git a/site/src/components/CopyMarkdownButton.tsx b/site/src/components/CopyMarkdownButton.tsx index 53d5d04a..08a50fb8 100644 --- a/site/src/components/CopyMarkdownButton.tsx +++ b/site/src/components/CopyMarkdownButton.tsx @@ -82,45 +82,50 @@ export default function CopyMarkdownButton({ className, style }: CopyMarkdownBut const ariaLabel = state.status === 'success' ? 'Copied' : 'Copy markdown to clipboard'; return ( - + + {state.status === 'success' ? 'Copied' : state.status === 'error' ? 'Error copying markdown' : ''} - - Copied - - - Error - - + ); } diff --git a/site/src/components/DialNav.tsx b/site/src/components/DialNav.tsx index 09a16edc..d7f0ebab 100644 --- a/site/src/components/DialNav.tsx +++ b/site/src/components/DialNav.tsx @@ -68,6 +68,8 @@ export default function DialNav({ left, right }: DialNavProps) { handleClick(e, left[0])} + aria-hidden="true" + tabIndex={-1} className="w-20 h-20 md:w-32 md:h-32 shrink-0 relative z-10 text-faded-black dark:text-manila-light [--fill:var(--color-manila-light)] dark:[--fill:var(--color-faded-black)]" > -