From af7e7a52a62e1380d44bd5983048565b3b1fb998 Mon Sep 17 00:00:00 2001
From: "roc+%cs.cmu.edu" <roc+%cs.cmu.edu>
Date: Sat, 17 Jul 2004 12:25:46 +0000
Subject: [PATCH] Bug 243720. Dragging stuff into menus should use screenX to
 avoid ambiguous semantics of clientX in popups. r+sr=neil

---
 xpfe/components/bookmarks/resources/bookmarksMenu.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/xpfe/components/bookmarks/resources/bookmarksMenu.js b/xpfe/components/bookmarks/resources/bookmarksMenu.js
index eb2028eb7e3d..42a3e0b8a468 100644
--- a/xpfe/components/bookmarks/resources/bookmarksMenu.js
+++ b/xpfe/components/bookmarks/resources/bookmarksMenu.js
@@ -212,15 +212,15 @@ var BookmarksMenu = {
       case "toolbarseparator":
       case "toolbarbutton":
         size = overButtonBoxObject.width;
-        coordValue = overButtonBoxObject.x;
+        coordValue = overButtonBoxObject.clientX;
         clientCoordValue = aEvent.clientX;
         break;
       case "menuseparator": 
       case "menu":
       case "menuitem":
         size = overButtonBoxObject.height;
-        coordValue = overButtonBoxObject.y-overParentBoxObject.y;
-        clientCoordValue = aEvent.clientY;
+        coordValue = overButtonBoxObject.screenY;
+        clientCoordValue = aEvent.screenY;
         break;
       default: return BookmarksUtils.DROP_ON;
     }