fix(theme-classic): Use native href for skip to content facebook#6411.

mturoci · Oct 11, 2022 · ee90a0b · ee90a0b
1 parent 3558a09
commit ee90a0b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/packages/docusaurus-theme-classic/src/theme/Layout/index.tsx b/packages/docusaurus-theme-classic/src/theme/Layout/index.tsx
@@ -42,6 +42,7 @@ export default function Layout(props: Props): JSX.Element {
       <Navbar />
 
       <div
+        id="main-wrapper"
         className={clsx(
           ThemeClassNames.wrapper.main,
           styles.mainWrapper,

diff --git a/packages/docusaurus-theme-classic/src/theme/SkipToContent/index.tsx b/packages/docusaurus-theme-classic/src/theme/SkipToContent/index.tsx
@@ -18,8 +18,10 @@ export default function SkipToContent(): JSX.Element {
       ref={containerRef}
       role="region"
       aria-label={translate({id: 'theme.common.skipToMainContent'})}>
-      {/* eslint-disable-next-line jsx-a11y/anchor-is-valid */}
-      <a href="#" className={styles.skipToContent} onClick={handleSkip}>
+      <a
+        href="#main-wrapper"
+        className={styles.skipToContent}
+        onClick={handleSkip}>
         <Translate
           id="theme.common.skipToMainContent"
           description="The skip to content label used for accessibility, allowing to rapidly navigate to main content with keyboard tab/enter navigation">