From 731d677da6d775716d8e07668d81a007033dc56e Mon Sep 17 00:00:00 2001
From: Jean-Luc Makiola <business@jeanlucmakiola.de>
Date: Mon, 13 Apr 2026 20:10:02 +0200
Subject: [PATCH] fix: shared setup items link to catalog instead of requiring
 auth

Items with a globalItemId now link to /global-items/:id (public) in
shared and public setup views. Items without a catalog link are not
clickable. Owner view behavior unchanged.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 src/client/components/ItemCard.tsx    | 19 +++++++++++++++----
 src/client/routes/setups/$setupId.tsx |  7 +++++++
 2 files changed, 22 insertions(+), 4 deletions(-)

diff --git a/src/client/components/ItemCard.tsx b/src/client/components/ItemCard.tsx
index ef7a19f..abbca67 100644
--- a/src/client/components/ItemCard.tsx
+++ b/src/client/components/ItemCard.tsx
@@ -25,6 +25,7 @@ interface ItemCardProps {
 	onRemove?: () => void;
 	classification?: string;
 	onClassificationCycle?: () => void;
+	linkTo?: string | null;
 }
 
 export function ItemCard({
@@ -46,19 +47,29 @@ export function ItemCard({
 	onRemove,
 	classification,
 	onClassificationCycle,
+	linkTo,
 }: ItemCardProps) {
 	const { weight, price } = useFormatters();
 	const navigate = useNavigate();
 	const openExternalLink = useUIStore((s) => s.openExternalLink);
 	const duplicateItem = useDuplicateItem();
 
+	const handleClick =
+		linkTo === null
+			? undefined
+			: () => {
+					if (linkTo) {
+						navigate({ to: linkTo });
+					} else {
+						navigate({ to: "/items/$itemId", params: { itemId: String(id) } });
+					}
+				};
+
 	return (
 		<button
 			type="button"
-			onClick={() =>
-				navigate({ to: "/items/$itemId", params: { itemId: String(id) } })
-			}
-			className="relative w-full text-left bg-white rounded-xl border border-gray-100 hover:border-gray-200 hover:shadow-sm transition-all overflow-hidden group"
+			onClick={handleClick}
+			className={`relative w-full text-left bg-white rounded-xl border border-gray-100 transition-all overflow-hidden group ${linkTo === null ? "cursor-default" : "hover:border-gray-200 hover:shadow-sm"}`}
 		>
 			{!onRemove && (
 				<span
diff --git a/src/client/routes/setups/$setupId.tsx b/src/client/routes/setups/$setupId.tsx
index 8e6abc9..8bf1ba0 100644
--- a/src/client/routes/setups/$setupId.tsx
+++ b/src/client/routes/setups/$setupId.tsx
@@ -381,6 +381,13 @@ function SetupDetailPage() {
 																})
 														: undefined
 												}
+												linkTo={
+													!showOwnerControls
+														? item.globalItemId
+															? `/global-items/${item.globalItemId}`
+															: null
+														: undefined
+												}
 											/>
 										))}
 									</div>