From 15ee2a2fb25b98bd9f85ade9f4985739da775d6c Mon Sep 17 00:00:00 2001 From: Max Schmitt Date: Tue, 4 Feb 2025 09:42:47 +0100 Subject: [PATCH] fix(core): a11y fix on mobile DropdownNavbarItem --- .../src/theme/NavbarItem/DropdownNavbarItem/index.tsx | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/packages/docusaurus-theme-classic/src/theme/NavbarItem/DropdownNavbarItem/index.tsx b/packages/docusaurus-theme-classic/src/theme/NavbarItem/DropdownNavbarItem/index.tsx index 4136410411aa5..a2b999585a53a 100644 --- a/packages/docusaurus-theme-classic/src/theme/NavbarItem/DropdownNavbarItem/index.tsx +++ b/packages/docusaurus-theme-classic/src/theme/NavbarItem/DropdownNavbarItem/index.tsx @@ -151,6 +151,10 @@ function DropdownNavbarItemMobile({ 'menu__link menu__link--sublist menu__link--sublist-caret', className, )} + // # hash permits to make the tag focusable in case no link target + // See https://github.com/facebook/docusaurus/pull/6003 + // There's probably a better solution though... + href={props.to ? undefined : '#'} {...props} onClick={(e) => { e.preventDefault();