Closed Bug 1741195 Opened 3 years ago Closed 3 years ago

Detaching an attachment leads to never-ending spinner

Categories

(Thunderbird :: FileLink, defect)

defect

Tracking

(thunderbird_esr91 unaffected, thunderbird95 unaffected)

RESOLVED WORKSFORME
Tracking Status
thunderbird_esr91 --- unaffected
thunderbird95 --- unaffected

People

(Reporter: rachel, Unassigned)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Firefox/91.0

Steps to reproduce:

Send yourself a message with a text file attachment. In the attachment list, right-click the attachment, select "Detach". Save the attachment some where.
Open the attachment area again.

Actual results:

The attachment shows a never-ending spinner.

Expected results:

The spinner shouldn't show. It's only for external attachments which are being fetched.

Alice, can you please find the regression.

Flags: needinfo?(alice0775)
Keywords: regression

Works fine in TB 95 beta, so it's a very recent regression. Instead of the spinner, you should see the icon belonging to the attachment type, like if it weren't detached.

I cannot reproduce the issue in ether Tb75.0b3 and Daily96.0a1(20211115103144) Windows10.

Open the attachment area again.

Right click on the attachment and choose Open. The attachment will be opened in my text editor as expected.

Flags: needinfo?(alice0775)
Assignee: nobody → john
Status: UNCONFIRMED → NEW
Ever confirmed: true
Regressed by: 1667652

I will fix it.

Component: Untriaged → FileLink

Backed out bug 1667652 so this is now resolved.

Assignee: john → nobody
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WORKSFORME
Attached image detach.PNG (deleted) —

I think you guys found a second bug in your STR. That should actually be the detached icon, not the default/standard icon, after the attachment has been detached. At least we have this in the code:
https://searchfox.org/comm-central/rev/fc1f4d8339c84577090fda2e716059074e11430b/mail/base/content/widgets/mailWidgets.js#1603-1604

This used to work in TB78 (see screenshot), but it no longer works like this in TB91.

Have we changed that deliberately, or should I file that as a new bug?

Detached and deleted are two different things. Only a deleted attachment will have the word "Deleted:" prepended.
https://searchfox.org/comm-central/rev/fc1f4d8339c84577090fda2e716059074e11430b/mailnews/mime/src/mimemult.cpp#245
There's a bug to localise that, bug 323109. As far as we can tell, everything works in 91.

Ah, correct, thanks for the explanation.

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: