bugzilla.mozilla.org will be intermittently unavailable on Saturday, March 24th, from 16:00 until 20:00 UTC.

[FEATURE] Tasks menu implementation

RESOLVED FIXED in mozilla1.0


19 years ago
14 years ago


(Reporter: daver, Assigned: Ben Goodger (use ben at mozilla dot org for email))


({helpwanted, verifyme})

helpwanted, verifyme
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)




(1 attachment)



19 years ago
The Tasks menu has the following items:

     Instant Messenger
     Security Advisor
     Address Book

According to German:

"Hi Dave:
Agree, the Menu Framework specification for 5.0 specifically mentions Security
to be a menu item in the Tasks menu, but in poisition
much lower than what you describe:

Instant Messenger
Address Book
Services A  >
Services B  >
Services C  >
Services D  >
Services E  >
Services F  >
Services G >
Services H  >
Security Info   Shift-I
Tools  >
Go To Window  >

Where did you find this?
+ G "


19 years ago
Assignee: michaell → german
Summary: [beta] Security Advisor should move down in task menu → [beta] need final task menu implementation

Comment 1

19 years ago
Changing Summary from "[beta] Security Advisor should move down in task menu" to
"[beta] need final task menu implementation".  Reassigning to German.  I agree
with the email from German that Dave copied into the bug.


19 years ago
Assignee: german → don

Comment 2

19 years ago
Don: we need help with implementing this menu as spec'd. Who is working on the
corss-app menus these days? I assume Matt would be the right person?

Comment 3

19 years ago
This was being handled by davidm.  I am guessing it still is.


19 years ago
Whiteboard: need to find owner for this (mcafee?)
Target Milestone: M14


19 years ago
Priority: P3 → P1
Summary: [beta] need final task menu implementation → Task Menu Implementation

Comment 4

19 years ago
Hmmmm ...


19 years ago
Assignee: don → matt
Summary: Task Menu Implementation → [FEATURE] Task Menu Implementation
Whiteboard: need to find owner for this (mcafee?)

Comment 5

19 years ago
Matt ...

Comment 6

19 years ago
*** Bug 15100 has been marked as a duplicate of this bug. ***

Comment 7

19 years ago
Reminder to nbaca to check 15100 when this bug is fixed.
QA Contact: claudius → sairuh
several of the Task items are in my realm. reassigning qa contact to me.

Comment 9

18 years ago
Chris, you are now the proud owner of the task menu/switcher for beta 1.  Aren't
you the lucky one.
Assignee: matt → mcafee

Comment 10

18 years ago
The task menu/switcher has to work for beta and contain the correct items. 
Nominating as a "beta1" blocker.
Keywords: beta1

Comment 11

18 years ago
Also see bug 24121 which details what we are doing for beta 1 in the Task Bar.
Getting everything right in this menu is not as critical as the Task Bar - 
Netcenter expects the Task Bar but is less worried about the menu for beta 1.  
We just need to make sure that this menu doesn't "suck".

This doesn't have to be a beta 1 blocker, but it should remain M14.

Comment 12

18 years ago
removed Don's beta1 nomination - Don OKed that.
Keywords: beta1

Comment 13

18 years ago
If Net2Phone is installed on the end user's machine, a TASK menu item for 
Net2Phone should be placed in the 2nd group where the Address Book is listed.  
List Net2Phone as the last component in this group.

Comment 14

18 years ago
Moving all UE/UI bugs to new component: User Interface: Design Feedback
UE/UI component will be deleted.
Component: UE/UI → User Interface: Design Feedback

Comment 15

18 years ago

Target Milestone: M14 → M15

Comment 16

18 years ago
Move to M16 for now ...
Target Milestone: M15 → M16


18 years ago
Target Milestone: M16 → M17
nominating for beta2 --however, i'm not even sure what the "must haves" are for
this menu. anyone? bueller?
Keywords: nsbeta2
OS: Windows NT → All
QA Contact: sairuh → mpt
Hardware: PC → All

Comment 18

18 years ago
Putting on [nsbeta2+][6/15] radar.
Whiteboard: [nsbeta2+][6/15]

Comment 19

18 years ago
Move to M19 target milestone.
Target Milestone: M17 → M19

Comment 20

18 years ago
I guess it's about time we fixed these up to look exactly like they're supposed to. Johng and German, I can't find specs for this 
stuff that have been updated since 00:00 Jan 1, 1970 (that's what the dates say :-), could you point some out? While you're at it 
specs for the taskbar(separate bug) would be cool too as I think things have shifted a little since PR1.

Comment 21

18 years ago
Assigning to German for input on what this menu should say
Assignee: mcafee → german

Comment 22

18 years ago
Cleaning up status whiteboard by marking beta2 minus (6/15 has psssed)

It sonuds like we need a spec to work towards... and I'm sure folks are too 
doomed to change much of this for beta2
Whiteboard: [nsbeta2+][6/15] → [nsbeta2-]

Comment 23

18 years ago
nsbeta3 to keep on radar.  Important polish issue.
Keywords: nsbeta3

Comment 24

18 years ago
JohnG I assume this is fixed - right?. Right the Tasks menu is implemented 
according to spec, except we have now a submenu "Privacy and Security" as decided 
elsewhere, as well as we decided to simply appending the list of windows to the 
last section of Tasks menu instead of having another submenu in order to make 
them more discoverable. Marking worksforme. Please reopen if you see the need or 
file new bug if we need to change/amend other stuff.
Last Resolved: 18 years ago
Resolution: --- → WORKSFORME

Comment 25

18 years ago
Created attachment 12324 [details]
Task Menu

Comment 26

18 years ago
I'm reopening the bug. Show me the spec. Currently, there are 3 implementations: The one outlined in this bug originally, the 
one in the recent attachment, and the one in the builds :-/. Also, the attachement and German's most recent comments 
contradict each other.
Resolution: WORKSFORME → ---

Comment 27

18 years ago
nav triage team:
nsbeta3+, P1
the current implementation is correct except that we need to add a link for
"Download" ...  German will clarify.
Whiteboard: [nsbeta2-] → [nsbeta2-][nsbeta3+]

Comment 28

18 years ago
This looks like a job for Timeless ...

For the record, the current implementation is: (as far as I can tell, coz 
Messenger is missing from the latest build for some reason)

  Address Book
  Privacy and Security  > [hideously complicated submenus here -- see bug 48860]
------------------------- ______________________
  Tools                 >|  History             |
-------------------------|  Import Utility      |
  {list}                 |  Java Console        |
  {of}                   |  JavaScript Console  |
  {open}                  """"""""""""""""""""""

Obvious problems with this:

* Bookmarks aren't present, meaning that there is no way to open the Bookmarks
  window from a non-Navigator window (bug 44524).

* The latest attachment shows `Auto-fill form', but that has been moved to the
  Edit menu (bug 42438, bug 48986). Meanwhile, I've just asked that `Forms
  Manager' be given its own main menu item, since it's really got nothing to do
  with privacy and security.

  Since the latter would just be taking up the real estate abandoned by the
  former, I assume there would be no objection to this? Please annotate bug 50494

* `Import Utility' should be named by what it does, not what it is, i.e.
  `Import/Export ...'

* Too many menu separators.
Depends on: 44524, 50494

Comment 29

18 years ago
Yes I can do everything except the following two mirracles:
a) Get german to make the spec reasonable (or change the spec at all)
b) Change the api.
Currently xul allows absolute positioning.  This is not a good idea.

Assuming I want something near:
  Address Book

I can guess that the number 6 is ok for positioning

My current menu has:
7IRC Chat [as you can see Chatzilla is already suffering because of this 
feature absense]
Someone else might have:

Privacy And Security
7IRC Chat

Until we change the api to be sane I can't truely fix all of the problems to 
match the requested spec.

Hi Ben, I'm sure you know who should address the insertion issues. Please CC 
them w/ comment.

For the moment, I think I could use java script to get the id's of the 
separators, and insert after them.
We use multiple <menupopup><!--s--></menupopup> For the tasks menu, And just 
combine them using some JS glue [The overlay writers specify which tasks menu 
they want to put something in], this solves the location functionality very 
easily and cleanly (except for js glue). 

<menu id="taskPopup" onload="compositeTaskPopup()">
<menupopup id="appSuite">
<menu id="navi" value="Navigator">
<menupopup id="appExtras">
<menu id="adbk" value="Address Book">
<menupopup id="appPSM">
<menu id="psm" value="Privacy and Security">
<menu id="tools"/>
<menupopup id="appWindows"/>

By default a separator will appear before and after each menupopup (except 
where it would be the first or last item in the menu).

Actually this proposal solves problems in File>New> and probably elsewhere.  If 
I can get the glue working I'll attach a demo patch.

Blake what do you think?
I also have a bunch of problems with the spec.

German I now have CVS. I'm going to make changes to the spec according to a 
proposal I sent you months ago.  If you have objections please send them to me 
w/in the week.

Hi Brendan. German is unresponsive. I'll forward a copy of my proposed changes.
Keywords: helpwanted
Whiteboard: [nsbeta2-][nsbeta3+] → [nsbeta2-][nsbeta3+][buggy spec WILL be corrected RSN]

Comment 30

18 years ago
PDT does not see what is critical to do here?  Looks like low profile polish.  
Moving to P3.  Adding [PDTP3]
Priority: P1 → P3

Comment 31

18 years ago
(Cosmetics only, but anyhow... )

Also, selecting the "Address Book" item if the mail component is not installed
does nothing. I guess it should not be rendered at all, or at least disabled?

Comment 32

18 years ago
I have worked on this and am almost ready to give it to people, my solution 
would also work for File>New>.

If I don't attach the patch to this bug by wednesday, someone should nag me.

Comment 33

18 years ago
timeless: some comments.
You need to add new files to makefile.win, makefile.mk, MANIFEST, and jar.mn 
files. It also needs to be verified that this works in the netscape commercial 

IMHO this is too risky for nsbeta3.

Comment 34

18 years ago
cc self

Comment 35

18 years ago
As of today, the first level of the menu on the commercial build looks fine with
2 possible changes.

Commerical build today (build 2000091408)looks like this:


- Navigator   Ctrl+1
- Mail      Ctrl+2
- Instant Messenger  Ctrl+3
- Composer   Ctrl+4
- Address Book   Ctrl+5
- Net2Phone
- Privacy and Security >
- Tools
(session history list)


Privacy and Security >
- Secuirty Manager
- Password Manager >
- Form Manager >
- Cookie Manager >
- Understanding Privacy

All of this, and the details within the sub-folders, is currently correct with 2
possible exceptions:

1)  The link to "Understanding Privacy" appears to be pointing to the wrong
document.  See bug 52674
2)  We need to add an item to this menu, specifically "Download" and this should
immediately follow "Tools" before the separator.  The url should be the
following:  http://home.netscape.com/bookmark/6_0/tdownload.html

Comment 36

18 years ago
- Address Book   Ctrl+5
- Net2Phone
- Privacy and Security >

That's wrong.

Net2Phone should not be near P&S.

Comment 37

18 years ago
I was unable to guarantee the menu item positions because
we can only specify absolute position of menu items (I think).
Specifically, AIM in the #3 slot pushes other elements down
but does NOT push them down for mozilla, so the absolute insertion
point is a moving target and makes Task menu ordering non-deterministic.
If we could use relative positioning or some dummy element
mechanism, that would be good.  Hyatt or ben?
Assignee: german → ben

Comment 38

18 years ago
Just to clarify the Netscape Task menu is supposed to look like this:



Instant Messenger


Address Book



Other installed components (like Chatzilla or ThemeBuilder)


Privacy and Security    >

 Tools      > 


Open Window 1

Open Window 2


Open Window n

This is also what McAfee had attempted to do, but as he describes the 

ordering problem was such that this could not be accomplished easily. 

The bug goes to Ben (cc'ing Hyatt) because either one of this guys might 

know of mighter ways to insert menus other than the above mentioned 

absolute positioning.                            
Whiteboard: [nsbeta2-][nsbeta3+][buggy spec WILL be corrected RSN] → [nsbeta2-][nsbeta3+]

Comment 39

18 years ago
I have a method that'd work, but it'd be to do some javascript sorting on a non 
rendered popupmenu before inserting into the real menu. I'll look into this 
It is impossible to force add on applications to occupy a given range. An add 
on, such as chatzilla, mailnews, AIM etc may insert any nodes it pleases into 
any node with an ID, at any index. 

What specifically is going wrong? 

Comment 41

18 years ago
Temporarily elevating to P2.

Is this a stop ship bug?  Do we want to move it to RTM?
Priority: P3 → P2

Comment 42

18 years ago
I had a recommended implementation for this, but I think mozilla killed my 
computer while i was distracted and had not submitted, i'll write the 
recommendation and proposed patch tonight.

sfraser: it'll be based on that code I asked you to check, so I need to know if 
that code worked on macos.

Comment 43

18 years ago
PDT thinks this is an nsbeta3- at this point. If you renominate for RTM, please
resummarize the critical issue at hand.
Whiteboard: [nsbeta2-][nsbeta3+] → [nsbeta2-][nsbeta3-]

Comment 44

18 years ago
[resummarizing so this bug is easier to find]
Summary: [FEATURE] Task Menu Implementation → [FEATURE] Tasks menu implementation

Comment 45

18 years ago
yeah id's might work.
Since this is coming in from a (possibly) commercial overlay,
we might want to have generically-named menu id stubs.
There is no M19
Target Milestone: M19 → mozilla0.6

Comment 47

18 years ago
There is no further feature work in mozilla0.6, either. Assume you meant 
Target Milestone: mozilla0.6 → mozilla0.9
What is wanted here, again? Trying to prune my .9 list of non-bookmarks related 

Comment 49

17 years ago
See also bug 32502.
out we go. 
Target Milestone: mozilla0.9 → mozilla1.0

Comment 51

17 years ago
Chaning the qa contact on these bugs to me. MPT will be moving to the 
owner of this component shortly. I would like to thank him for all his hard 
work as he moves roles in mozilla.org...Yada, Yada, Yada...
QA Contact: mpt → zach

Comment 52

17 years ago
What is still needed here?  Ben has asked a few times, with no answer.  
Closing, file new bugs for specific issues.
Last Resolved: 18 years ago17 years ago
Resolution: --- → FIXED

Comment 53

17 years ago
the problem reported here is that (a) our implementation doesn't match the 
spec and (b) items tend to land out of their ranges [examples of things that 
fall incorrectly include chatzilla and some commercial items]. it has been 
mentioned that we now have insertafter and insertbefore which should probably 
work to fix this bug.

I've seen relatively recent reports of (b) so i'd say it isn't fixed. but 
Keywords: nsbeta2, nsbeta3 → verifyme
Whiteboard: [nsbeta2-][nsbeta3-]


16 years ago
Component: User Interface Design → Browser-General
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.