From 57ddef4f9419e30478c367843df4d98c064888cc Mon Sep 17 00:00:00 2001 From: "gavin%gavinsharp.com" Date: Mon, 5 May 2008 20:03:37 +0000 Subject: [PATCH] Bug 431535: Static item's context menu in bookmark menu is messed up, Marco Bonardo , r=mano, a=mconnor git-svn-id: svn://10.0.0.236/trunk@251218 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/browser/components/places/content/menu.xml | 4 ++-- mozilla/browser/components/places/content/toolbar.xml | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/mozilla/browser/components/places/content/menu.xml b/mozilla/browser/components/places/content/menu.xml index e6f9f56cbec..6307b5381b7 100755 --- a/mozilla/browser/components/places/content/menu.xml +++ b/mozilla/browser/components/places/content/menu.xml @@ -911,8 +911,8 @@ var selectedNode = this.selectedNode; if (selectedNode) { var popupNode = document.popupNode; - if (popupNode == popupNode.parentNode._emptyMenuItem) { - // If (empty) menuitem is selected the insertion point + if (!popupNode.node) { + // If a static menuitem is selected the insertion point // is inside the folder, at the end. folderId = PlacesUtils.getConcreteItemId(selectedNode); orientation = Ci.nsITreeView.DROP_ON; diff --git a/mozilla/browser/components/places/content/toolbar.xml b/mozilla/browser/components/places/content/toolbar.xml index b2c4d73018c..69efae46c58 100755 --- a/mozilla/browser/components/places/content/toolbar.xml +++ b/mozilla/browser/components/places/content/toolbar.xml @@ -392,8 +392,8 @@ var selectedNode = this.selectedNode; if (selectedNode) { var popupNode = document.popupNode; - if (popupNode == popupNode.parentNode._emptyMenuItem) { - // If (empty) menuitem is selected the insertion point + if (!popupNode.node) { + // If a static menuitem is selected the insertion point // is inside the folder, at the end. folderId = PlacesUtils.getConcreteItemId(selectedNode); orientation = Ci.nsITreeView.DROP_ON;