40401 - Need to cleanup View | Messages

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect, P3)

Description

[scottputterman]

Since I'm not continuing with the feature of View Unread Threads, I need to do 
some cleanup of the View | Messages menuitem. Currently there are two working 
views: All (threaded and flat) and Unread (flat only).  The work that needs to 
be done is:

1.  Remove extra menu items.
2.  Make it so that Unread doesn't work in threaded mode and that when you are 
in the unread view that you can't get into threaded mode.
3.  Remember which view you are in per folder.

All of this is done and is working in my tree.

Comment 1

[scottputterman]

nominating for beta2.  It would be nice to have this cleanup in there now.

Comment 2

[leger]

Putting on [nsbeta2+][6/15] radar.

Comment 3

[selmer (gone)]

Target nominated bugs to M16.  Please update as necessary.

Comment 4

[scottputterman]

I like a reprieve on this one if possible.  I had this ready to check in and 
code reviewed and then the tree was basically shut down for most of Thursday and 
Friday.  

Why I'd like to check this in:  Basically, when I finished up View | Unread 
messages, I was hoping to get an extension on View | Threads with Unread 
messages. So I kept working on that and never cleaned up View | Unread.  The way 
it is in the current build is that you can choose options that don't work and 
will do strange things to your message view.  This bug is to disable those 
things that don't work.  In addition, the view you are in is pretty worthless if 
it doesn't remember it from session to session, so this bug also covers 
remembering it.

Comment 5

[Jim Roskind]

Cleaning up status whiteboard by changing to beta2 minus (6/15 has passed)

... but based on the above comment, you might possibly re-nominate for beta2 by 
deleting the whole whiteboard.

We have a lot of critical stuff needed for beta2... so be sure your in good 
shape on the current stopper.

Thanks,

Jim

Comment 6

[scottputterman]

I just don't care anymore.  Sometimes playing this game gets tiring. Let people 
get into views that mess them up during beta2.  I'll see if I can save this code 
that's been sitting in my tree and possibly get it in by beta3. I don't plan on 
rewriting it if I lose it or it's hard to merge.  Moving to M18 and nominating 
for beta3.

Comment 7

[scottputterman]

By now you'd thought I'd have learned that if I'm unhappy with a decision such 
as this I should wait for 1/2 a day to reply instead of replying right away and 
having to make an apology later. :-)

Anyway, I was just frustrated because I have these changes in my tree.  I will 
take them out and if we decide this is needed for nsbeta3 I will find a way to 
merge them back in.

Comment 8

[Ben Bucksch (:BenB)]

[OT] putterman,
I liked to read your feelings and can even completely follow them.

We're humans and do have feelings. If done right, articulating them often helps
*avoiding* escalations.

Comment 9

[lchiang]

+ per mail triage

Comment 10

[robinf]

Adding myself to cc: list.

Comment 11

[scottputterman]

fix checked in.  Removed unimplemented items.  Disable Unread mode when 
threaded.  Disable threaded when in unread mode.

Comment 12

[scottputterman]

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

Comment 13

[fenella]

Linux (2000-08-16-09 M18)
Win32 (2000-08-16-10 M18)
Mac (2000-08-16-10 M18)
This works as expected as described by Scott, which is:
Removed unimplemented items.  Disable Unread mode when threaded.  Disable
threaded when in unread mode.

Comment 14

[scottputterman]

Removing the mail1 keyword.