From 80160cb2cb07f2b77c593755e72c593cd5490d90 Mon Sep 17 00:00:00 2001
From: kei <keletrh@gmail.com>
Date: Thu, 11 Jun 2026 10:23:10 +0200
Subject: [PATCH 1/2] test(mail): cover attachment detail tooltips

---
 .../singlemailviewer.component.spec.ts        | 27 ++++++++++++++++---
 1 file changed, 24 insertions(+), 3 deletions(-)

diff --git a/src/app/mailviewer/singlemailviewer.component.spec.ts b/src/app/mailviewer/singlemailviewer.component.spec.ts
index d4bc44f93..7078d7932 100644
--- a/src/app/mailviewer/singlemailviewer.component.spec.ts
+++ b/src/app/mailviewer/singlemailviewer.component.spec.ts
@@ -33,13 +33,17 @@ import { MatIconTestingModule } from '@angular/material/icon/testing';
 import { MatLegacyMenuModule as MatMenuModule } from '@angular/material/legacy-menu';
 import { MatLegacyRadioModule as MatRadioModule } from '@angular/material/legacy-radio';
 import { MatToolbarModule } from '@angular/material/toolbar';
-import { MatLegacyTooltipModule as MatTooltipModule } from '@angular/material/legacy-tooltip';
+import {
+  MatLegacyTooltip as MatTooltip,
+  MatLegacyTooltipModule as MatTooltipModule
+} from '@angular/material/legacy-tooltip';
 import { MatLegacySnackBarModule as MatSnackBarModule } from '@angular/material/legacy-snack-bar';
 import { FormsModule } from '@angular/forms';
 import { CommonModule } from '@angular/common';
 import { HttpClient } from '@angular/common/http';
 import { Observable, of } from 'rxjs';
 import { NoopAnimationsModule } from '@angular/platform-browser/animations';
+import { By } from '@angular/platform-browser';
 import { AvatarBarComponent } from './avatar-bar.component';
 import { RouterTestingModule } from '@angular/router/testing';
 import { Router } from '@angular/router';
@@ -153,12 +157,14 @@ describe('SingleMailViewerComponent', () => {
               attachments: [
                 {
                   filename: 'test.jpg',
-                  contentType: 'image/jpeg'
+                  contentType: 'image/jpeg',
+                  size: 1536
                 },
                 {
                   filename: 'test2.png',
                   contentType: 'image/png',
-                  content: Uint8Array.from([1, 2, 3, 4, 5, 6, 7, 8])
+                  content: Uint8Array.from([1, 2, 3, 4, 5, 6, 7, 8]),
+                  size: 8
                 }
               ]
             }));
@@ -231,6 +237,21 @@ describe('SingleMailViewerComponent', () => {
       expect(component.mailObj.attachments[1].downloadURL.indexOf('blob:')).toBe(0);
     }));
 
+  it('shows attachment details in the attachment tile tooltip', fakeAsync(() => {
+      fixture.detectChanges();
+      component.messageId = 22;
+      fixture.detectChanges();
+      tick(1);
+      fixture.detectChanges();
+
+      const attachmentTile = fixture.debugElement.query(By.css('mat-grid-tile'));
+      const tooltip = attachmentTile.injector.get(MatTooltip);
+
+      expect(tooltip.message).toContain('test.jpg');
+      expect(tooltip.message).toContain('image/jpeg');
+      expect(tooltip.message).toContain(component.mailObj.attachments[0].sizeDisplay);
+    }));
+
   describe('mailto: link interceptor', () => {
     let messageContentsElement: HTMLElement;
     let mailtoLink: HTMLAnchorElement;

From 52fbd9deb590afb59cb7abc819e086bf139b9b5d Mon Sep 17 00:00:00 2001
From: kei <keletrh@gmail.com>
Date: Thu, 11 Jun 2026 10:23:26 +0200
Subject: [PATCH 2/2] feat(mail): show attachment detail tooltips

---
 src/app/mailviewer/singlemailviewer.component.html |  2 ++
 src/app/mailviewer/singlemailviewer.component.ts   | 11 +++++++++++
 2 files changed, 13 insertions(+)

diff --git a/src/app/mailviewer/singlemailviewer.component.html b/src/app/mailviewer/singlemailviewer.component.html
index 5eaf9debb..9ff3d6f6c 100644
--- a/src/app/mailviewer/singlemailviewer.component.html
+++ b/src/app/mailviewer/singlemailviewer.component.html
@@ -391,6 +391,8 @@
           <mat-grid-tile
             *ngFor="let att of mailObj.attachments; let i=index"
             (click)="openAttachment(att)"
+            [matTooltip]="attachmentDetails(att)"
+            [attr.aria-label]="attachmentDetails(att)"
             style="max-width: 150px">
             <mat-grid-tile-header>
               <span class="string-ellipsis">{{att.sizeDisplay}}</span>
diff --git a/src/app/mailviewer/singlemailviewer.component.ts b/src/app/mailviewer/singlemailviewer.component.ts
index 1bb81b032..334008910 100644
--- a/src/app/mailviewer/singlemailviewer.component.ts
+++ b/src/app/mailviewer/singlemailviewer.component.ts
@@ -354,6 +354,17 @@ export class SingleMailViewerComponent implements OnInit, DoCheck, AfterViewInit
     }
   }
 
+  attachmentDetails(attachment: any): string {
+    const filename = attachment.filename || 'Unnamed attachment';
+    const contentType = attachment.contentType || 'Unknown type';
+    const sizeDisplay = attachment.sizeDisplay
+      || (typeof attachment.size === 'number'
+        ? MessageTableRowTool.formatBytes(attachment.size, 2)
+        : 'Unknown size');
+
+    return `${filename} - ${contentType} - ${sizeDisplay}`;
+  }
+
   public toggleHtml(event) {
     event.preventDefault();