220656 - Changing folder doesn't reset the View drop down list to All

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, enhancement)

Description

[Pierre Antonini]

User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.0; Q312461; .NET CLR 1.0.3705)
Build Identifier: Mozilla Thunderbird 0.3 (20030923)

When changing from one folder to another, the View drop down list should be 
reset to display all messages of the newly selected folder.

Reproducible: Always

Steps to Reproduce:
1.
2.
3.

Comment 1

[Daniel Beckham]

IMHO, the behavior should be one of two things, a) maintain a View state for
each folder, or b) maintain the same view for all folders, possibly across
Thunderbird sessions.

Resetting the view state each time you move to a new folder and never saving the
last state the folder was in, seems counter-intuitive.

Comment 2

[Nik the Greek]

I'd like to suggest a view-per-folder policy. This is what OE does and it's,
IMHO, far more usefull than a 'global' view.

Comment 3

[Scott MacGregor]

I'll flag this for 0.9. David and I are now convinced we should just save this
value per folder....

Comment 4

[Scott MacGregor]

Attachment: the fix

remember the view as a per folder setting

Comment 5

[Scott MacGregor]

Comment on attachment 163760 [details] [diff] [review]
the fix

1) It's unclear from the diff, but the changes to msgMail3PaneWindow.js are
inside the folder loaded notification.

2) When porting this patch to the trunk, I need to account for the
getUint32Property API changes.

3) When porting to the trunk I'll also change mailnews\msgMail3PaneWindow.js

4) We aren't loading the remembered view correctly for a virtual folder. But we
didn't do it correctly before this patch either so I'll figure that out
separately from this bug.

Comment 6

[Scott MacGregor]

fixed branch and trunk

Comment 7

[[e-mail]]

Two nits:
* If you use -p for your diffs it will add (in this case) var folderListener = {
at the end of the @@ line that begins the diff.
* Save yourself a parseInt and let XPConnect do it for you.

Comment 8

[Mike Cowperthwaite]

*** Bug 239783 has been marked as a duplicate of this bug. ***