[Pkg-mozext-commits] [itsalltext] 302/459: Fixed the context menu positioning in FF3, yay!

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository itsalltext.

commit 5de9165b65d23f08f149ade8f65d1603d93ef937
Author: docwhat at gerf.org <docwhat at gerf.org>
Date:   Sat Jan 12 03:26:39 2008 -0500

    Fixed the context menu positioning in FF3, yay!
---
 src/chrome/content/cacheobj.js | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/src/chrome/content/cacheobj.js b/src/chrome/content/cacheobj.js
index 554cced..5ffe6b2 100644
--- a/src/chrome/content/cacheobj.js
+++ b/src/chrome/content/cacheobj.js
@@ -532,21 +532,23 @@ CacheObj.prototype.onContext = function(event) {
      * See Mozilla bugs:
      * https://bugzilla.mozilla.org/show_bug.cgi?id=287357
      * https://bugzilla.mozilla.org/show_bug.cgi?id=291083
+     *
+     * This is actually fixed in FF3 by replacing it with something
+     * sane....openPopup()
      */
     var cobj = ItsAllText.getCacheObj(event.target);
     var popup = ItsAllText.rebuildMenu(cobj.uid);
-    /*
-    popup.showPopup(popup,
-                    event.screenX, event.screenY,
-                    'context', null, null);
-    */
+
     if (popup.openPopup) {
-        popup.openPopup(null, 'after_pointer', null, null, true, null);
+        /* FF3 breath of sanity. */
+        popup.openPopup(cobj.button, 'end_before',
+                        0, cobj.gumdrop_height,
+                        true, false);
     } else {
         document.popupNode = popup;
         popup.showPopup(document.documentElement,
                         event.screenX, event.screenY,
-                        'popup', null, null);
+                        'popup', false, false);
     }
     event.stopPropagation();
     return false;

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/itsalltext.git