254946 - "compact folders" disabled after emptying the trash (because trash folder is deselected automatically, no folder selected)

Status: RESOLVED WORKSFORME

(SeaMonkey :: MailNews: Message Display, defect)

Description

[YoaR]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a2) Gecko/20040714
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a2) Gecko/20040714

"Compact Folders" from File menu is disabled after emptying the trash with the
Trash folder selected.

Reproducible: Always
Steps to Reproduce:
1. Move an email to the trash (delete it)
2. Go to the Trash folder
3. Do "Empty Trash" from File menu
4. Compact Folders from File menu is now disabled, you have to select another
folder in order to be enabled again.
Actual Results:  
Compact Folders gets disabled

Expected Results:  
Lets you click on Compact Folders menu item.

Comment 1

[Mike Cowperthwaite]

Reproduced with 
  Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8a3) Gecko/20040806

Confirming; updating platform

Comment 2

[Mike Cowperthwaite]

A closer examination reveals that this occurs because if the Trash folder is
selected, on being emptied it is deselected (the focus remains on the pane, but 
no item is selected in the tree).

xref bug 192870.

Comment 3

[Felix Miata]

Comment 2 is ancient behavior. Even though quite annoying, I've never looked for
an existing bug on it. Old Netscape versions (< v4) automatically compacted when
emptying trash, which is what Moz should do too.

Comment 4

[OstGote!]

(In reply to comment #3)
> Old Netscape versions (< v4) automatically compacted when
> emptying trash, which is what Moz should do too.

That is Bug 61960.

Comment 5

[Andreas Kunz]

When a local Trash folder (as opposed to an IMAP one) is emptied, this occurs by
deleting it and creating a new one, see:
http://lxr.mozilla.org/seamonkey/source/mailnews/local/src/nsLocalMailFolder.cpp#951

I guess this is why there is no folder selected after emptying the Trash folder
and the solution would be just to select the newly created folder. Don't know
where this should happen and if it is possible in nsLocalMailFolder.cpp.
Probably rather some layers above.

Comment 6

[Andreas Kunz]

Bug 280804 is approx. the same for Thunderbird (it also includes the "I deleted
an arbitrary folder" case which is valid for Seamonkey, too). Leaving both open
(although the "empty trash" code is shared) because the solution might very well
take place in the forked UI layer.

Comment 7

[Wayne Mery (:wsmwk)]

WFM in version 3
Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1.5) Gecko/20091127 Shredder/3.0.1pre

please reopen if you can reproduce in v3