1729279 - Attachment file sizes no longer included in the list of attachments printed at the end of a message

Status: VERIFIED FIXED

(MailNews Core :: Printing, defect, P4)

Description

[Thomas D. (:thomas8)]

Attachment: Screenshot 1: TB 78 vs. TB 91: no longer prints attachment sizes

+++ This bug was initially created as a clone of Bug #1727055 +++

Steps to reproduce:

1. Print a mail with attachments
2. Check attachment list at end of printout for file sizes

Actual results:

The sizes of the attachments are no longer included in the list of attachments when printing an email in 91.

Expected results:

Attachment sizes should be printed as they always were up to TB 78. Attachment size is a relevant and helpful detail in many ways.

Comment 1

[Geoff Lankow (:darktrojan)]

Attachment: Bug 1729279 - Add attachment size to emitted HTML for printing. r=mkmelin

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/c2e1acd467e0
Add attachment size to emitted HTML for printing. r=mkmelin

Comment 3

[Geoff Lankow (:darktrojan)]

Comment on attachment 9239718 [details]
Bug 1729279 - Add attachment size to emitted HTML for printing. r=mkmelin

[Approval Request Comment]
Regression caused by (bug #): bug 1711408
User impact if declined: printed mail doesn't show the size of attachments
Testing completed (on c-c, etc.): 3 day on c-c
Risk to taking this patch (and alternatives if risky): low

Comment 4

[Wayne Mery (:wsmwk)]

Comment on attachment 9239718 [details]
Bug 1729279 - Add attachment size to emitted HTML for printing. r=mkmelin

[Triage Comment]
Approved for beta

Comment 5

[Rob Lemley [:rjl]]

Thunderbird 93.0b2:
https://hg.mozilla.org/releases/comm-beta/rev/540459c45286

Comment 6

[WaltS48 [:walts48]]

Verified using Thunderbird 93.0b2 on Windows 10.

Comment 7

[Geoff Lankow (:darktrojan)]

Comment on attachment 9239718 [details]
Bug 1729279 - Add attachment size to emitted HTML for printing. r=mkmelin

[Approval Request Comment]
Regression caused by (bug #): bug 1711408
User impact if declined: printed mail doesn't show the size of attachments
Testing completed (on c-c, etc.): in 93.0b2
Risk to taking this patch (and alternatives if risky): low

Comment 8

[Wayne Mery (:wsmwk)]

Comment on attachment 9239718 [details]
Bug 1729279 - Add attachment size to emitted HTML for printing. r=mkmelin

[Triage Comment]
Approved for esr91

Comment 9

[Rob Lemley [:rjl]]

Thunderbird 91.1.2:
https://hg.mozilla.org/releases/comm-esr91/rev/f2d1714c605b