537275 - Add some element IDs to navigator to assist porting of Firefox extensions.

Status: RESOLVED FIXED

(SeaMonkey :: UI Design, enhancement)

Description

[Philip Chee]

Sync some element IDs from browser.xul to make it easier for Firefox extensions to overlay SeaMonkey.

Comment 1

[Philip Chee]

Attachment: Patch v1.0

+        <menuitem id="historyMenuUp" 
Firefox doesn't have this menu item, but for consistency I'm adding one to all the menu items in this area.

-        <menuitem label="&historyCmd.label;" accesskey="&historyCmd.accesskey;" oncommand="toHistory()" key="key_gotoHistory"/>
-        <menuseparator hidden="true"/>
+        <menuitem id="menu_showAllHistory"
+                  label="&historyCmd.label;"
+                  accesskey="&historyCmd.accesskey;"
+                  oncommand="toHistory()"
+                  key="key_gotoHistory"/>

+        <menuseparator id="startHistorySeparator" hidden="true"/>
+        <menuseparator id="endHistorySeparator" hidden="true"/>
Several Firefox extensions use insertbefore/insertafter these menuseparators in their overlays.

-          aParent.lastChild.hidden = (count == 0);
If an extension appends a menu item to the Go menu, this logic hides the wrong menu item.

+          var startHistory = document.getElementById("startHistorySeparator");
+          var endHistory = document.getElementById("endHistorySeparator");
+          startHistory.hidden = (count == 0);
+          endHistory.hidden = (endHistory == aParent.lastChild);

Comment 2

[Philip Chee]

-function createMenuItem( aParent, aIndex, aLabel)
+function createMenuItem(aParent, aIndex, aLabel)

-function createRadioMenuItem( aParent, aIndex, aLabel, aChecked)
+function createRadioMenuItem( aParent, aAnchor, aIndex, aLabel, aChecked)

Aargh. I'll fix these whitespace nits in the next patch after reviews.

Comment 4

[[e-mail]]

Comment on attachment 419575 [details] [diff] [review]
Patch v1.0

>+                createRadioMenuItem(aParent, endHistory, j, entry.title, j==index);
Speaking of spacing nits, might as well fix the spacing around the ==

>     if (aChecked==true)
And if you're fixing all the spacing anyway, then this can be improved!

Comment 5

[[e-mail]]

Comment on attachment 419575 [details] [diff] [review]
Patch v1.0

Bah, after all that, I forgot to tick the boxes...

Comment 6

[Philip Chee]

Attachment: Patch v1.1 Fix whitespace nits. r=neil sr=neil [Checkin: Comment 8]

> (From update of attachment 419575 [details] [diff] [review])
>>+                createRadioMenuItem(aParent, endHistory, j, entry.title, j==index);
> Speaking of spacing nits, might as well fix the spacing around the ==
Fixed.

>>     if (aChecked==true)
> And if you're fixing all the spacing anyway, then this can be improved!
Fixed. Plus a few more whitespace nits around this line.

Comment 7

[Philip Chee]

Interdiff:
<https://bugzilla.mozilla.org/attachment.cgi?oldid=419575&action=interdiff&newid=419830&headers=1>

Comment 8

[Serge Gautherie (:sgautherie)]

Comment on attachment 419830 [details] [diff] [review]
Patch v1.1 Fix whitespace nits. r=neil sr=neil
[Checkin: Comment 8]


http://hg.mozilla.org/comm-central/rev/78e022d07edc