From 9f87bf18c2f7d417c0904a8409419c8e1666afb3 Mon Sep 17 00:00:00 2001
From: Sami Mazouz <sychocouldy@gmail.com>
Date: Fri, 25 Oct 2024 16:00:08 +0100
Subject: [PATCH] fix: header dropdown item might not be linkable

---
 .../js/src/forum/components/HeaderListItem.tsx   | 16 +++++++++++++---
 framework/core/less/forum/HeaderList.less        |  1 +
 2 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/framework/core/js/src/forum/components/HeaderListItem.tsx b/framework/core/js/src/forum/components/HeaderListItem.tsx
index d2ec789b19..8b382673d9 100644
--- a/framework/core/js/src/forum/components/HeaderListItem.tsx
+++ b/framework/core/js/src/forum/components/HeaderListItem.tsx
@@ -13,7 +13,7 @@ export interface IHeaderListItemAttrs extends ComponentAttrs {
   content: string;
   excerpt: string;
   datetime?: Date;
-  href: string;
+  href?: string | null;
   onclick?: (e: Event) => void;
   actions?: Mithril.Children;
 }
@@ -22,8 +22,18 @@ export default class HeaderListItem<CustomAttrs extends IHeaderListItemAttrs = I
   view(vnode: Mithril.Vnode<CustomAttrs, this>) {
     const { avatar, icon: iconName, content, excerpt, datetime, href, className, onclick, actions, ...attrs } = vnode.attrs;
 
+    const Tag = href ? Link : 'button';
+
     return (
-      <Link className={classList('HeaderListItem', className)} href={href} external={href.includes('://')} onclick={onclick}>
+      <Tag
+        className={classList('HeaderListItem', className, {
+          'Button--ua-reset': Tag === 'button',
+        })}
+        href={href}
+        external={href?.includes('://')}
+        onclick={onclick}
+        {...attrs}
+      >
         {avatar}
         <Icon name={iconName} className="HeaderListItem-icon" />
         <span className="HeaderListItem-title">
@@ -33,7 +43,7 @@ export default class HeaderListItem<CustomAttrs extends IHeaderListItemAttrs = I
         </span>
         <div className="HeaderListItem-actions">{actions}</div>
         <div className="HeaderListItem-excerpt">{excerpt}</div>
-      </Link>
+      </Tag>
     );
   }
 }
diff --git a/framework/core/less/forum/HeaderList.less b/framework/core/less/forum/HeaderList.less
index a2585fd261..79adc904c4 100644
--- a/framework/core/less/forum/HeaderList.less
+++ b/framework/core/less/forum/HeaderList.less
@@ -168,6 +168,7 @@
   align-items: baseline;
   row-gap: 1px;
   column-gap: 6px;
+  cursor: pointer;
 
   // Prevent outline overflowing parent
   .add-keyboard-focus-ring-offset(-1px);