Closed Bug 19147 Opened 25 years ago Closed 21 years ago

View|Folders needs to be implemented.

Categories

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

Product:
SeaMonkey
Component:
MailNews: Message Display
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: waqar, Assigned: ssu0262)

References

Details

(Keywords: access, polish, Whiteboard: 1 day)

Attachments

(8 files)

proposed patch although View|Message/Folders may belong under View|Show
4.90 KB, patch
Details | Diff | Splinter Review
Patch that also hides the message splitter as per jglick
8.44 KB, patch
Details | Diff | Splinter Review
Revised patch - I missed some files out :-( Apologies for SPAM.
15.18 KB, patch
Details | Diff | Splinter Review
Still trying to figure out what should do what...
7.35 KB, text/html
Details
At last! This is it!
21.54 KB, patch
Details | Diff | Splinter Review
Spoke to soon - diffed wrong files :-( Hope these are OK!
23.49 KB, patch
Details | Diff | Splinter Review
Yet more SPAM - bitrot this time, sorry.
24.23 KB, patch
Details | Diff | Splinter Review
fix typo and bitrot in previous patch
24.46 KB, patch
Details | Diff | Splinter Review 
Connect to an IMAP server and then select View->Folders to hide the folders, it
does not hide them. The check mark is removed but the folders are still
displayed.
Assignee: phil → putterman
Summary: Selecting View->Folder does not work → [FEATURE] View|Folders, View|Message do not work
View | Message also doesn't work. Probably not a big surprise at this point :-)

Reassigning to putterman
Status: NEW → ASSIGNED
Blocks: 10791
OS: Linux → All
QA Contact: lchiang → fenella
Hardware: PC → All
View | Message is covered by http://bugzilla.mozilla.org/show_bug.cgi?id=14106
Target Milestone: M16
*** Bug 21811 has been marked as a duplicate of this bug. ***
View|Message now works.
Whiteboard: 1 day
Syncing priority with marketing.  Moving to P2 to connote "In" for beta2.
Priority: P3 → P2
Target Milestone: M16 → M17
[FEATURE] bugs past M16 are OUT for this release.  Marking M20.  If you disagree 
with this action, please help me explain it to the PDT.
Target Milestone: M17 → M20
What is left for this bug?  We can't have menu items which don't work.  Fenella 
- can you check and summarize here?
changing milestone.  This is going to be simple UI.  It's only called a feature 
because it used to be a line item on a schedule.  This will be simple.
Target Milestone: M20 → M18
for the moment nominating for nsbeta3 and polish.  I can't remember the outcome 
of our meeting. Currently there's a My Sidebar menu but not a folders menuitem.  
I know that nbaca's been pulling for this.

changing summary to refer to what's left of this bug.
Keywords: nsbeta3, polish
Summary: [FEATURE] View|Folders, View|Message do not work → View|Folders needs to be implemented.
Keywords: mail2
Keywords: mail2 → mail6
mail triage marking [nsbeta3-]
Whiteboard: 1 day → [nsbeta3-]1 day
Sorry for the extra email. Removing mail6 keyword.
Keywords: mail6
Target Milestone: M18 → ---
I would have a go at this but I don't know what access key to use :-(
F is taken by Folder Character Coding.
for now, make it use a dtd and "" as the access key, i think ew can get intl to 
relinquish f later. [just 2 dtd changes]
Keywords: nsbeta3 → access
Whiteboard: [nsbeta3-]1 day → 1 day
What will be the result of the menu item?  View | Sidebar just removes the
Sidebar but doesn't close the folder pane/sidebar splitter.  Will View | Folders
just remove the Folder pane or will it close the whole splitter?  If it just
removes the folder pane, perhaps we need logic that says if both View | Folders
and View | Sidebar are unchecked, close the splitter.
Is Sidebar part of Mail Folders or its own entity?  If it is its own entity, 
unchecking Mail Folders would remove the Mail Folders but My Sidebar would 
remain and take up the whole left pane.  

As putterman mentioned, would be nice if either Mail Folders or Sidebar was 
already unchecked, and then the user unchecked the other one, the whole left 
splitter went away.
As for the mnemonic, I strongly believe that "Folder Character Coding" should be 
removed since it opens the exact same dialog as "Edit --> (folder) properties. 
Bug 71972.
View Folders is a lot easier on the alternate 3 pane layout because the sidebar 
and folders have their own splitters!
On the alternate layout, everything works as expected.
But the best that I have been able to do on the default layout is this:
View Folders collapses the folder splitter.
If the folder splitter is collapsed then View My Sidebar could be disabled.

There were two problems that stumped me.

If the sidebar and folders are visible and the folders are hidden from the menu
then they can only be shown from the menu. But the alternate layout folders and
the message pane are collapsed rather than hidden and can be shown using the grippy.

If the sidebar and folders are visible and the vertical splitter is collapsed
then what should the menu items do? If the sidebar and folders are hidden from
the menu then what should the vertical splitter do?

The only way I can think of to resolve these problems is to make all view
options always hide the splitters are well instead of just My Sidebar.
>If the sidebar and folders are visible and the folders are hidden from the menu
>then they can only be shown from the menu. But the alternate layout folders and
>the message pane are collapsed rather than hidden and can be shown using the 
>grippy.

Using the "View" menu item (My Sidebar or Folder Pane) should not just collapse 
the splitter/grippy but should remove the item (My Sidebar or Folder 
Pane) completely.

>If the sidebar and folders are visible and the vertical splitter is collapsed
>then what should the menu items do? 

If the splitters are only collasped (and not removed), the menu items should 
remove them completely.  Selecting the menu item again would cause the splitter 
to reappear in its collapsed state.

>If the sidebar and folders are hidden from
>the menu then what should the vertical splitter do?

If the the sidebar and/or folders are hidden/turned off from the menu item, the 
the vertical splitter should not be visible.


jglick@netscape.com wrote:
> Using the "View" menu item (My Sidebar or Folder Pane) should not just collapse
> the splitter/grippy but should remove the item (My Sidebar or Folder Pane)
> completely.

Does this apply to the message pane as well? Currently there is code that checks
to see if the message pane is collapsed and it would have to be modified to
check to see if it was removed as well.
Ideally, i would think the message pane should work the same as well (currently 
it doesn't, the splitter is always visible).
N.B. This patch doesn't address the sidebar on the standard layout (there is no
problem with the alternate layout). Might need some help from sidebar folks.
Otherwise does anyone want to update this bug appropriately?
CCing sidebar folks for review of changes to sidebarOverlay files
Message Pane:
- I agree that the splitter should not be visible when I deselect the msg page
in VIew|Show.
- Also, currently, the subject of the selected message is still shown in the
titlebar of the window, which is confusing (in the taskbar, it looks like
standalone msg window, because the "INBOX" comes so late that it's invisible).
(Lucky, the message doesn't seem to load, when I select in.) If you get a
chance, it would be cool, if you could fix that, too.
I think, both problems are a regression.

Ben, this bug is about the folder pane. If you don't like the subject appearing
in the title when the message pane is collapsed you should IMHO file a separate bug.
Keywords: patch, review
QA Contact: fenella → laurel
Neil, you've been doing all of the work on this, can I reassign it to you?
reassigning to ssu
Assignee: putterman → ssu
Status: ASSIGNED → NEW
If attachment 56566 [details] [diff] [review] to bug 105542 is checked in this will make this bug 
trivial to fix.
Depends on: 105542
Sigh, this will have to wait until bug 123336 is fixed - there's no point hiding
the folders while doing that breaks mail and news...
Depends on: 123336
The menuitem doesn't exist any more. WONTFIX or WFM.
No longer valid.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → INVALID
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: