From 00f16b04ab2d55ff35d1b7e40ae2857e2fea9a12 Mon Sep 17 00:00:00 2001
From: Bill Thornton <thornbill@users.noreply.github.com>
Date: Fri, 28 Mar 2025 12:52:52 -0400
Subject: [PATCH] Add default href value to LinkButton

---
 src/apps/stable/routes/user/settings/index.tsx | 4 ----
 src/elements/emby-button/LinkButton.tsx        | 2 +-
 2 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/apps/stable/routes/user/settings/index.tsx b/src/apps/stable/routes/user/settings/index.tsx
index d8cf6bda1b..e392118bd5 100644
--- a/src/apps/stable/routes/user/settings/index.tsx
+++ b/src/apps/stable/routes/user/settings/index.tsx
@@ -182,7 +182,6 @@ const UserSettingsPage: FC = () => {
 
                         {appHost.supports('clientsettings') && (
                             <LinkButton
-                                href='#' // Required for focus in TV layout
                                 onClick={shell.openClientSettings}
                                 className='clientSettings listItem-border'
                                 style={{
@@ -288,7 +287,6 @@ const UserSettingsPage: FC = () => {
 
                             {appHost.supports('multiserver') && (
                                 <LinkButton
-                                    href='#' // Required for focus in TV layout
                                     onClick={Dashboard.selectServer}
                                     className='selectServer listItem-border'
                                     style={{
@@ -309,7 +307,6 @@ const UserSettingsPage: FC = () => {
                             )}
 
                             <LinkButton
-                                href='#' // Required for focus in TV layout
                                 onClick={Dashboard.logout}
                                 className='btnLogout listItem-border'
                                 style={{
@@ -330,7 +327,6 @@ const UserSettingsPage: FC = () => {
 
                             {appHost.supports('exitmenu') && (
                                 <LinkButton
-                                    href='#' // Required for focus in TV layout
                                     onClick={appHost.exit}
                                     className='exitApp listItem-border'
                                     style={{
diff --git a/src/elements/emby-button/LinkButton.tsx b/src/elements/emby-button/LinkButton.tsx
index 77bf39a3f0..359bc0d6b0 100644
--- a/src/elements/emby-button/LinkButton.tsx
+++ b/src/elements/emby-button/LinkButton.tsx
@@ -18,7 +18,7 @@ interface LinkButtonProps extends DetailedHTMLProps<AnchorHTMLAttributes<HTMLAnc
 const LinkButton: React.FC<LinkButtonProps> = ({
     className,
     isAutoHideEnabled,
-    href,
+    href = '#', // The href must have a value to be focusable in the TV layout
     target,
     onClick,
     children,