thunderbird doesn't have the account/folder list under the Go menu

RESOLVED FIXED in Thunderbird 3.3a2

Status

--
enhancement
RESOLVED FIXED
11 years ago
8 years ago

People

(Reporter: dds, Assigned: dds)

Tracking

unspecified
Thunderbird 3.3a2

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments, 4 obsolete attachments)

(Assignee)

Description

11 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9) Gecko/2008052906 Firefox/3.0
Build Identifier: 3.0a2pre (2008061403) and older

Thunderbird doesn't have a Go - Folder menu command.  This hinders navigating to a folder without a mouse, and is therefore an accessibility issue.   The underlying functionality (menu-based folder navigation) is already part of the code, as one can see by selecting "Move To" on the context menu of a message from the message list pane.  Such a feature used to be there in SeaMonkey and older Mozilla versions.  The Nostalgy plugin doesn't solve the problem, because it doesn't provide hierarchical navigation. 

Reproducible: Always

Steps to Reproduce:
Select the "Go" menu". 
Actual Results:  
 A "Folder" command is missing.  

Expected Results:  
A "Folder" submenu should appear.  Each element of the "Folder" submenu is the name of a folder, which can again be a submenu, if the folder contains subfolders.  This functionlity exists in SeaMonkey.  The submenu's entries are already available in the "Move To" and "Copy To" message context menu commands.
SeaMonkey implementation was bug 21344 - xref bug 349767. The SM menu looks like this:

 -----------------
 Next
 Previous
 Forward
 Back
 -----------------
 Mail Start Page
 -----------------
 Work Mail >
 News & Blogs >
 news.mozilla.org >
 ------------------ 

Hm, we do have a lot of room under the Go menu. I think I like it.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → All
Hardware: PC → All
Summary: Missing Go - Folder menu command → thunderbird doesn't have the account/folder list under the Go menu
(Assignee)

Comment 2

11 years ago
Created attachment 330842 [details] [diff] [review]
Patch against version 2.0.0.14 (20080421) that implements the proposed change

The implementation could be further improved by removing the intermediate Folder menu thus saving an additional keystroke.
(Assignee)

Comment 3

11 years ago
Created attachment 331406 [details] [diff] [review]
Patch tested on version 2.0.0.16 (20080708) that implements the proposed change

The previously submitted patch had incomplete patches for mailWindowOverlay.xul
Attachment #330842 - Attachment is obsolete: true
(Assignee)

Comment 4

8 years ago
Created attachment 490469 [details] [diff] [review]
Patch for Thunderbird 3.0.4

This is a patch against the files distributed with Thunderbird 3.0.4.  It has been tested and it works correctly.
Attachment #331406 - Attachment is obsolete: true
(Assignee)

Comment 5

8 years ago
Created attachment 490473 [details] [diff] [review]
Patch for Thunderbird 3.0.9 source code

This is an untested hand-crafted patch against the Thunderbird 3.0.9 source code.
Diomidis, thank you very much for the paches. To be accepted in to Thunderbird your patch will need a few things :

1) It needs to be against trunk/tip of the tree (see below)
2) as It touches the UI, you'll need to get ui review on it (providing a screen-shot helps the review in terms of speed)
3) You'll need to get code reviews as well.


You'll find the complete set of rules at https://developer.mozilla.org/en/comm-central .

Following the docs at https://developer.mozilla.org/En/Developer_Guide/Source_Code/Getting_comm-central should let you create a patch against trunk/tip.
(Assignee)

Comment 7

8 years ago
Created attachment 490619 [details] [diff] [review]
Patch against trunk 50d164d40d14
Attachment #490473 - Attachment is obsolete: true
(Assignee)

Comment 8

8 years ago
Created attachment 490620 [details]
User interface: popup menu

This shows the selection of a folder by pressing ALT-G-O-L-A-S (Go - Folder - Local Folders - AFolder - Subfolder) on a Windows-7 system.

Updated

8 years ago
Attachment #490619 - Flags: ui-review?(clarkbw)
(Assignee)

Comment 9

8 years ago
Created attachment 490624 [details]
User interface: after the selection

After selecting the subfolder the menu closes and the folder is selected on the folder selection pane.
(Assignee)

Comment 10

8 years ago
(In reply to comment #6)
> Diomidis, thank you very much for the paches. To be accepted in to Thunderbird
> your patch will need a few things :
> 
> 1) It needs to be against trunk/tip of the tree (see below)
Done - see comment 7.

> 2) as It touches the UI, you'll need to get ui review on it (providing a
> screen-shot helps the review in terms of speed)
I added two screen shots - see comments 8 and 9.  I don't know how to arrange a UI menu.

> 3) You'll need to get code reviews as well.
When I first added the patches (comments 4 and 5) I sent email to clarkbw asking for a code review.
Comment on attachment 490619 [details] [diff] [review]
Patch against trunk 50d164d40d14

Looks pretty good to me
Attachment #490619 - Flags: ui-review?(clarkbw) → ui-review+
Marco, do you think this would help?
(Assignee)

Updated

8 years ago
Attachment #490619 - Flags: review?(mkmelin+mozilla)
Attachment #490619 - Flags: review?(bugzilla)
Comment on attachment 490619 [details] [diff] [review]
Patch against trunk 50d164d40d14

No need to request two reviews one is enough.
Attachment #490619 - Flags: review?(mkmelin+mozilla)
Attachment #490619 - Flags: review?(bwinton)
Attachment #490619 - Flags: review?(bugzilla)
Comment on attachment 490619 [details] [diff] [review]
Patch against trunk 50d164d40d14

>+++ b/mail/base/content/mailWindowOverlay.xul	Mon Nov 15 10:24:39 2010 +0200
>@@ -1332,16 +1332,30 @@
>           <menuseparator id="goNextSeparator"/>
>+          <menu id="goFolderMenu"
>+		    label="&folderMenu.label;"
>+		    accesskey="&folderMenu.accesskey;"
>+                    oncommand="gFolderTreeView.selectFolder(event.target._folder, true)">

Please don't use tabs, and line the attributes up with the "id" attribute.

>+            <menupopup id="menu_GoFolderPopup"
>+                 type="folder"
>+                 showFileHereLabel="true"
>+                 fileHereLabel="&thisFolder.label;"
>+                 fileHereAccessKey="&thisFolder.accesskey;"
>+                 showRecent="true"
>+                 recentLabel="&contextMoveCopyMsgRecentMenu.label;"
>+                 recentAccessKey="&contextMoveCopyMsgRecentMenu.accesskey;"/>

And these should be lines up with the "id" attribute, too.

But other than that, I like it, so r=me with those nits fixed.

(As a side note, we would usually need tests for this, but I think the folder-widget tests probably do a pretty good job of testing this functionality.  Of course, if you'ld like to learn MozMill, and write some tests for this, I would be more than happy to review them!  :)

Thanks,
Blake.
Attachment #490619 - Flags: review?(bwinton) → review+
Keywords: checkin-needed

Comment 15

8 years ago
I think we need a correctly formatted patch before adding the checkin-needed flag, just to be on the safe side. But once there's a patch with the nits fixed, please add checkin-needed.
Keywords: checkin-needed
(Assignee)

Comment 16

8 years ago
Created attachment 491849 [details] [diff] [review]
Nits fixed following :bwinton code review

This modifies the original patch (attachment 490619 [details] [diff] [review]) corrected per :bwinton review.
Attachment #490619 - Attachment is obsolete: true
Attachment #491849 - Flags: review?(bwinton)
Comment on attachment 491849 [details] [diff] [review]
Nits fixed following :bwinton code review

You didn't actually need to re-ask for a review, cause you only fixed the things I asked you to fix, but r=me anyways. :)

Thanks,
Blake.
Attachment #491849 - Flags: review?(bwinton) → review+
Keywords: checkin-needed
Checked in:
http://hg.mozilla.org/comm-central/rev/ff0495607a53
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Assignee: nobody → dds
Keywords: checkin-needed
Target Milestone: --- → Thunderbird 3.3a2
You need to log in before you can comment on or make changes to this bug.