Bug 674208 - Add a Properties context menu option to open the embedded properties pane in Bookmarks Manager. r=IanN a=Ratty SEAMONKEY_2_5b3_BUILD1 SEAMONKEY_2_5b3_RELEASE
authorJens Hatlak <jh@junetz.de>
Wed, 12 Oct 2011 18:55:46 +0200
changeset 8899 062b443d5a8e2598ebabeb200ca6b2868a51ac50
parent 8898 cd566b95b6967245617d6b62a7e6b948320e54fd
child 8903 e7c127f06ea87fdb7ee2c0b2446194bb9a955412
child 8905 32e01434d527cc2ec701acafa5bcf0be7bf1fbd8
push id176
push userjh@junetz.de
push dateWed, 12 Oct 2011 17:09:21 +0000
treeherdercomm-beta@062b443d5a8e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersIanN, Ratty
bugs674208
Bug 674208 - Add a Properties context menu option to open the embedded properties pane in Bookmarks Manager. r=IanN a=Ratty
suite/common/bookmarks/bookmarksManager.js
suite/common/places/controller.js
--- a/suite/common/bookmarks/bookmarksManager.js
+++ b/suite/common/bookmarks/bookmarksManager.js
@@ -84,20 +84,16 @@ var PlacesOrganizer = {
       view.selection.select(0);
 
     this._content.focus();
 
     // Set up the search UI.
     PlacesSearchBox.init();
 
     window.addEventListener("AppCommand", this, true);
-
-    // remove the "Properties" context-menu item, we've our own details pane
-    document.getElementById("placesContext")
-            .removeChild(document.getElementById("placesContext_show:info"));
   },
 
   QueryInterface: function PO_QueryInterface(aIID) {
     if (aIID.equals(Components.interfaces.nsIDOMEventListener) ||
         aIID.equals(Components.interfaces.nsISupports))
       return this;
 
     throw Components.results.NS_NOINTERFACE;
--- a/suite/common/places/controller.js
+++ b/suite/common/places/controller.js
@@ -277,17 +277,21 @@ PlacesController.prototype = {
       break;
     case "placesCmd_new:livemark":
       this.newItem("livemark");
       break;
     case "placesCmd_new:separator":
       this.newSeparator();
       break;
     case "placesCmd_show:info":
-      this.showBookmarkPropertiesForSelection();
+      var dd = document.getElementById("detailsDeck");
+      if (dd)
+        dd.collapsed = false;
+      else
+        this.showBookmarkPropertiesForSelection();
       break;
     case "placesCmd_moveBookmarks":
       this.moveSelectedBookmarks();
       break;
     case "placesCmd_reload":
       this.reloadSelectedLivemark();
       break;
     case "placesCmd_sortBy:name":