Bug 804258 - the social sidebar should appear under View->Sidebar. r=…

…gavin
dothq · Oct 30, 2012 · ab1aa13 · ab1aa13
1 parent d8a3b43
commit ab1aa13
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/browser/base/content/browser-menubar.inc b/browser/base/content/browser-menubar.inc
@@ -210,6 +210,10 @@
                               observes="viewHistorySidebar"
                               label="&historyButton.label;"
                               accesskey="&historySidebarCmd.accesskey;"/>
+                    <menuitem id="menu_socialSidebar"
+                              type="checkbox"
+                              autocheck="false"
+                              command="Social:ToggleSidebar"/>
                   </menupopup>
                 </menu>
                 <menuseparator/>

diff --git a/browser/base/content/browser-social.js b/browser/base/content/browser-social.js
@@ -103,6 +103,9 @@ let SocialUI = {
     kbMenuitem.setAttribute("label", label);
     kbMenuitem.setAttribute("accesskey", accesskey);
 
+    // The View->Sidebar menu.
+    document.getElementById("menu_socialSidebar").setAttribute("label", Social.provider.name);
+
     SocialToolbar.init();
     SocialShareButton.init();
     SocialSidebar.init();