Bug 696731 - Properties context menu option in Bookmarks Manager does nothing when the embedded properties pane is already open. Focus the Name field. r=Neil a=IanN
authorJens Hatlak <jh@junetz.de>
Fri, 04 Nov 2011 23:47:16 +0100
changeset 9338 8067ee8641fc48074fee46d3f045417a0c674c43
parent 9337 f0c7a724baade856a744a3995daa33069386b0f5
child 9339 832b2312c94e66d89a39da3242076b6da8fc2e61
push idunknown
push userunknown
push dateunknown
reviewersNeil, IanN
bugs696731
Bug 696731 - Properties context menu option in Bookmarks Manager does nothing when the embedded properties pane is already open. Focus the Name field. r=Neil a=IanN
suite/common/places/controller.js
--- a/suite/common/places/controller.js
+++ b/suite/common/places/controller.js
@@ -278,20 +278,22 @@ PlacesController.prototype = {
     case "placesCmd_new:livemark":
       this.newItem("livemark");
       break;
     case "placesCmd_new:separator":
       this.newSeparator();
       break;
     case "placesCmd_show:info":
       var dd = document.getElementById("detailsDeck");
-      if (dd)
+      if (dd) {
         dd.collapsed = false;
-      else
+        document.getElementById("editBMPanel_namePicker").focus();
+      } else {
         this.showBookmarkPropertiesForSelection();
+      }
       break;
     case "placesCmd_moveBookmarks":
       this.moveSelectedBookmarks();
       break;
     case "placesCmd_reload":
       this.reloadSelectedLivemark();
       break;
     case "placesCmd_sortBy:name":