From 56ebe5728094d80af0782609140c88766844ea35 Mon Sep 17 00:00:00 2001
From: "roc+%cs.cmu.edu" <roc+%cs.cmu.edu>
Date: Sat, 5 May 2007 04:37:00 +0000
Subject: [PATCH] Bug 243720. Dragging stuff into menus should use screenX to
 avoid ambiguous semantics of clientX in popups. r=timeless,sr=neil

---
 suite/common/bookmarks/bookmarksMenu.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/suite/common/bookmarks/bookmarksMenu.js b/suite/common/bookmarks/bookmarksMenu.js
index eb2028eb7e3..a048d201c48 100644
--- a/suite/common/bookmarks/bookmarksMenu.js
+++ b/suite/common/bookmarks/bookmarksMenu.js
@@ -219,8 +219,8 @@ var BookmarksMenu = {
       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;
     }