From 1f76fa6c72025f05840c52848b7e021029f6331e Mon Sep 17 00:00:00 2001
From: Taylor Ho <taylorkmho@gmail.com>
Date: Tue, 28 Jul 2026 08:24:01 -0700
Subject: [PATCH] refactor(previews): remove preview-image caption and type
 label from cards
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Remove the "Automatic preview from {domain}" gradient overlay on preview
  images; simplify the img alt text to "Preview from {domain}"
- Drop the "· {typeLabel}" suffix (PR/issue/repo/file/link/etc.) from the
  card metadata row — the provider name and logo already convey the source,
  so the row now shows just the provider
- typeLabel stays in the SupportedLinkPreview model since it still powers
  the accessible aria-label ("Open GitHub PR: ...")

Signed-off-by: Taylor Ho <taylorkmho@gmail.com>
Co-authored-by: Taylor Ho <taylorkmho@gmail.com>
Signed-off-by: Taylor Ho <taylorkmho@gmail.com>
---
 desktop/src/shared/ui/link-preview-attachment.tsx | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/desktop/src/shared/ui/link-preview-attachment.tsx b/desktop/src/shared/ui/link-preview-attachment.tsx
index 8c81da33b..2a4d75bff 100644
--- a/desktop/src/shared/ui/link-preview-attachment.tsx
+++ b/desktop/src/shared/ui/link-preview-attachment.tsx
@@ -131,13 +131,10 @@ export function LinkPreviewAttachment({
         {showImage ? (
           <div className="relative -mx-3 -mt-2.5 w-[calc(100%+1.5rem)] overflow-hidden border-b border-border/70 bg-muted">
             <img
-              alt={`Automatic preview from ${preview.imageDomain}`}
+              alt={`Preview from ${preview.imageDomain}`}
               className="aspect-[1.91/1] w-full object-cover"
               src={preview.imageDataUrl ?? undefined}
             />
-            <div className="absolute inset-x-0 bottom-0 bg-gradient-to-t from-black/70 to-transparent px-3 pb-2 pt-6 text-2xs text-white">
-              Automatic preview from {preview.imageDomain}
-            </div>
           </div>
         ) : null}
         {!showImage ? (
@@ -148,8 +145,6 @@ export function LinkPreviewAttachment({
         <AttachmentContent className={showImage ? "w-full" : undefined}>
           <div className="truncate text-xs font-medium leading-4 text-muted-foreground">
             {preview.provider}
-            <span aria-hidden="true"> · </span>
-            {preview.typeLabel}
           </div>
           <AttachmentTitle>{preview.title}</AttachmentTitle>
         </AttachmentContent>