From 63eedafb1c3a0994777ee023171ebdd08fab3cc0 Mon Sep 17 00:00:00 2001
From: j-erler <j.erler90@gmx.de>
Date: Thu, 24 Sep 2026 00:16:13 +0200
Subject: [PATCH] Allowed the attachments dialog to hide the remove button

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 .../Dialogs/ReviewAttachmentsDialog.razor     | 35 +++++++++++++------
 .../Dialogs/ReviewAttachmentsDialog.razor.cs  | 15 ++++++--
 2 files changed, 37 insertions(+), 13 deletions(-)

diff --git a/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor b/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor
index 3528d5fc..8e11ef54 100644
--- a/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor	
+++ b/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor	
@@ -9,7 +9,14 @@
                       Disabled="@this.IsZoneDisabled"
                       Context="isDropTarget">
         <MudJustifiedText Typo="Typo.body1" Class="mb-3">
-            @T("Here you can see all attached files. Files that can no longer be found (deleted, renamed, or moved) are marked with a warning icon and a strikethrough name. You can remove any attachment using the trash can icon.")
+            @if (this.CanRemove)
+            {
+                @T("Here you can see all attached files. Files that can no longer be found (deleted, renamed, or moved) are marked with a warning icon and a strikethrough name. You can remove any attachment using the trash can icon.")
+            }
+            else
+            {
+                @T("Here you can see all attached files. Files that can no longer be found (deleted, renamed, or moved) are marked with a warning icon and a strikethrough name.")
+            }
         </MudJustifiedText>
 
         @if (this.CanAttach)
@@ -66,11 +73,14 @@
                                                    OnClick="@(() => this.InvestigateFile(fileAttachment))"/>
                                 </MudTooltip>
                             
-                                <MudTooltip Text="@T("Remove this attachment.")" Placement="Placement.Bottom">
-                                    <MudIconButton Icon="@Icons.Material.Filled.Delete"
-                                                   Color="Color.Error"
-                                                   OnClick="@(() => this.DeleteAttachment(fileAttachment))"/>
-                                </MudTooltip>
+                                @if (this.CanRemove)
+                                {
+                                    <MudTooltip Text="@T("Remove this attachment.")" Placement="Placement.Bottom">
+                                        <MudIconButton Icon="@Icons.Material.Filled.Delete"
+                                                       Color="Color.Error"
+                                                       OnClick="@(() => this.DeleteAttachment(fileAttachment))"/>
+                                    </MudTooltip>
+                                }
                             </MudToolBar>
                         </MudStack>
                     }
@@ -93,11 +103,14 @@
                                                Color="Color.Primary"
                                                Disabled="true"/>
                             
-                                <MudTooltip Text="@T("Remove this attachment.")" Placement="Placement.Bottom">
-                                    <MudIconButton Icon="@Icons.Material.Filled.Delete"
-                                                   Color="Color.Error"
-                                                   OnClick="@(() => this.DeleteAttachment(fileAttachment))"/>
-                                </MudTooltip>
+                                @if (this.CanRemove)
+                                {
+                                    <MudTooltip Text="@T("Remove this attachment.")" Placement="Placement.Bottom">
+                                        <MudIconButton Icon="@Icons.Material.Filled.Delete"
+                                                       Color="Color.Error"
+                                                       OnClick="@(() => this.DeleteAttachment(fileAttachment))"/>
+                                    </MudTooltip>
+                                }
                             </MudToolBar>
                         </MudStack>
                     }
diff --git a/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor.cs b/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor.cs
index daccf3c6..1f68c076 100644
--- a/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor.cs	
+++ b/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor.cs	
@@ -38,6 +38,16 @@ public partial class ReviewAttachmentsDialog : MSGComponentBase
     [Parameter]
     public Func<bool>? IsAttachingUnavailable { get; set; }
 
+    /// <summary>
+    /// Whether the user may remove attachments here.
+    /// </summary>
+    /// <remarks>
+    /// False when this dialog gathers the attachments of several messages at once: a removal would
+    /// reach into a message that was already sent, and nothing here could tell which one.
+    /// </remarks>
+    [Parameter]
+    public bool CanRemove { get; set; } = true;
+
     [Inject]
     private IDialogService DialogService { get; set; } = null!;
 
@@ -120,11 +130,12 @@ public partial class ReviewAttachmentsDialog : MSGComponentBase
 
     private async Task PathsDropped(List<string> paths) => await this.AttachPathsAsync(paths);
 
-    public static async Task<HashSet<FileAttachment>> OpenDialogAsync(IDialogService dialogService, HashSet<FileAttachment> documentPaths, Func<List<string>, Task<IReadOnlyList<FileAttachment>>>? attachPaths = null, Func<bool>? isAttachingUnavailable = null)
+    public static async Task<HashSet<FileAttachment>> OpenDialogAsync(IDialogService dialogService, HashSet<FileAttachment> documentPaths, Func<List<string>, Task<IReadOnlyList<FileAttachment>>>? attachPaths = null, Func<bool>? isAttachingUnavailable = null, bool canRemove = true)
     {
         var dialogParameters = new DialogParameters<ReviewAttachmentsDialog>
         {
-            { x => x.DocumentPaths, documentPaths }
+            { x => x.DocumentPaths, documentPaths },
+            { x => x.CanRemove, canRemove },
         };
 
         if (attachPaths is not null)