From e5448d9a96f3a006e1866a73785607c2dcc09f7c Mon Sep 17 00:00:00 2001
From: Alexey Pyltsyn <lex61rus@gmail.com>
Date: Mon, 19 Oct 2020 18:17:57 +0300
Subject: [PATCH] fix(v2): fix error with required href attr of link in mobiles
 (#3607)

---
 .../src/theme/NavbarItem/DefaultNavbarItem.tsx                 | 1 +
 packages/docusaurus/src/client/exports/Link.tsx                | 3 ++-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/packages/docusaurus-theme-classic/src/theme/NavbarItem/DefaultNavbarItem.tsx b/packages/docusaurus-theme-classic/src/theme/NavbarItem/DefaultNavbarItem.tsx
index f6c60e4f5e..8790809062 100644
--- a/packages/docusaurus-theme-classic/src/theme/NavbarItem/DefaultNavbarItem.tsx
+++ b/packages/docusaurus-theme-classic/src/theme/NavbarItem/DefaultNavbarItem.tsx
@@ -184,6 +184,7 @@ function NavItemMobile({
         'menu__list-item--collapsed': collapsed,
       })}>
       <NavLink
+        role="button"
         className={navLinkClassNames(className, true)}
         {...props}
         onClick={() => {
diff --git a/packages/docusaurus/src/client/exports/Link.tsx b/packages/docusaurus/src/client/exports/Link.tsx
index c8ff63b237..cada960b34 100644
--- a/packages/docusaurus/src/client/exports/Link.tsx
+++ b/packages/docusaurus/src/client/exports/Link.tsx
@@ -146,7 +146,8 @@ function Link({
     // eslint-disable-next-line jsx-a11y/anchor-has-content
     <a
       href={targetLink}
-      {...(!isInternal && {target: '_blank', rel: 'noopener noreferrer'})}
+      {...(targetLinkUnprefixed &&
+        !isInternal && {target: '_blank', rel: 'noopener noreferrer'})}
       {...props}
     />
   ) : (