#109598, "tasks overlays should use insertbefore, insertafter instead of position"

#81237, "IRC Chat has bizarre menu position",

Patches by nnooiissee@yahoo.com

r=andreww, sr=jst

extensions/irc/xul/content/chatzillaOverlay.xul

@@ -19,11 +19,12 @@
 
 <menupopup id="taskPopup">
   <menuitem
-        position="3"
         label="&ircCmd.label;"
         accesskey="&ircCmd.accesskey;"
         key="key_irc"
-        command="Tasks:IRC"/>
+        command="Tasks:IRC"
+        id="tasksMenuIRC"
+        insertbefore="tasksMenuEditor"/>
 </menupopup>
 
 <statusbarpanel id="component-bar">

mailnews/base/resources/content/mailTasksOverlay.xul

@@ -67,8 +67,8 @@
   </statusbarpanel>
   
   <menupopup id="taskPopup">
-    <menuitem position="2" accesskey="&messengerCmd.accesskey;" label="&messengerCmd.label;"  key="key_mail" command="Tasks:Mail"/>
-    <menuitem position="4" label="&addressBookCmd.label;" accesskey="&addressBookCmd.accesskey;" key="key_addressbook" command="Tasks:AddressBook"/>
+    <menuitem accesskey="&messengerCmd.accesskey;" label="&messengerCmd.label;"  key="key_mail" command="Tasks:Mail" id="tasksMenuMail" insertafter="tasksMenuNavigator"/>
+    <menuitem label="&addressBookCmd.label;" accesskey="&addressBookCmd.accesskey;" key="key_addressbook" command="Tasks:AddressBook" id="tasksMenuAddressBook" insertafter="tasksMenuEditor"/>
   </menupopup>
   
 </overlay>         

xpfe/communicator/resources/content/tasksOverlay.xul

@@ -41,8 +41,8 @@
         </template>
 
         <menupopup id="taskPopup">
-          <menuitem label="&navigatorCmd.label;" accesskey="&navigatorCmd.accesskey;" key="key_navigator" command="Tasks:Navigator"/>
-          <menuitem label="&editorCmd.label;" accesskey="&editorCmd.accesskey;" key="key_editor" command="Tasks:Editor" />
+          <menuitem label="&navigatorCmd.label;" accesskey="&navigatorCmd.accesskey;" key="key_navigator" command="Tasks:Navigator" id="tasksMenuNavigator"/>
+          <menuitem label="&editorCmd.label;" accesskey="&editorCmd.accesskey;" key="key_editor" command="Tasks:Editor" id="tasksMenuEditor"/>
 
           <!-- Overlays get stuffed in here. -->