starred/webmail
1
0
Fork 0
mirror of https://github.com/bulwarkmail/webmail.git synced 2026-11-05 00:14:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: leave the message size out of the reader header

Browse Source
This commit is contained in:
Linus Rath
2026-11-04 06:37:59 +01:00
parent f828b6dffe
commit 8184b75196
2 changed files with 10 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
components/email/__tests__/reader-spam-trash-actions.test.tsx
+7
View File
@@ -103,3 +103,10 @@ describe('reader toolbar in the Inbox', () => {
});
});
describe('reader header', () => {
it('shows no message size (the details have it)', async () => {
render(<EmailViewer email={{ ...message('inbox'), size: 2150 } as Email} isLoading={false} mailboxes={mailboxes} selectedMailbox="inbox" currentMailboxRole="inbox" />);
await act(async () => { await new Promise((r) => setTimeout(r, 10)); });
expect(document.body.textContent).not.toContain('2.1 KB');
});
});
components/email/email-viewer.tsx
+3 -12
View File
@@ -5055,16 +5055,12 @@ ${printWatermark ? printWatermark.html : ''}
</div>
)}
{/* Date/time on the right of subject row - hidden on mobile, shown next to sender.
Mountain View puts it on the sender's row instead, and shows no size. */}
Mountain View puts it on the sender's row instead. The size is
in the details. */}
<div className={cn("hidden flex-shrink-0 text-end", !mvViewer && "sm:block")}>
<span className="text-xs lg:text-sm text-muted-foreground whitespace-nowrap">
{formatDateTime(emailDisplayDate(email), timeFormat, { weekday: 'short', year: 'numeric', month: 'short', day: 'numeric' })}
</span>
{email.size > 0 && (
<div className="text-xs text-muted-foreground">
<bdi>{formatFileSize(email.size)}</bdi>
</div>
)}
</div>
</div>
</div>
@@ -5415,18 +5411,13 @@ ${printWatermark ? printWatermark.html : ''}
</button>
</div>
</div>
{/* Date/time + size on the right (mobile) */}
{/* Date/time on the right (mobile); the size is in the details */}
<div className="sm:hidden flex-shrink-0 text-end ms-2">
<span className="text-xs text-muted-foreground whitespace-nowrap">
{mvMobileViewer
? formatDate(emailDisplayDate(email))
: formatDateTime(emailDisplayDate(email), timeFormat, { weekday: 'short', year: 'numeric', month: 'short', day: 'numeric' })}
</span>
{email.size > 0 && !mvMobileViewer && (
<div className="text-xs text-muted-foreground">
<bdi>{formatFileSize(email.size)}</bdi>
</div>
)}
</div>
{/* Mountain View phone: reply from the top of the message, not
only from the row under a long body (#1475). */}