From dbb07937f0c04402ab3c539c40fac9f6725acf8b Mon Sep 17 00:00:00 2001
From: Dan Stillman <dstillman@zotero.org>
Date: Wed, 17 Dec 2008 10:14:12 +0000
Subject: [PATCH] Fix for attachment/note tag popup not displaying in Firefox 3
 if there wouldn't be enough space on screen -- now auto-adjusts, though
 possibly with some artifacts after closing on OS X

---
 chrome/content/zotero/bindings/noteeditor.xml | 22 ++++++++++++++++---
 1 file changed, 19 insertions(+), 3 deletions(-)

diff --git a/chrome/content/zotero/bindings/noteeditor.xml b/chrome/content/zotero/bindings/noteeditor.xml
index fb4307b5cf..debf120663 100644
--- a/chrome/content/zotero/bindings/noteeditor.xml
+++ b/chrome/content/zotero/bindings/noteeditor.xml
@@ -389,7 +389,15 @@
 				<body>
 					<![CDATA[
 						this.id('tags').reload();
-						this.id('tagsPopup').showPopup(this.id('tagsLabel'),-1,-1,'popup');
+						if (this.id('tagsPopup').openPopupAtScreen) {
+							var x = this.boxObject.screenX;
+							var y = this.boxObject.screenY;
+							this.id('tagsPopup').openPopupAtScreen(x, y, false);
+						}
+						// Deprecated method for Firefox 2
+						else {
+							this.id('tagsPopup').showPopup(this.id('tagsLabel'), -1, -1, 'popup');
+						}
 					]]>
 				</body>
 			</method>
@@ -402,7 +410,7 @@
 						if (!v || v == "") {
 							v = "[" + Zotero.getString('pane.item.noteEditor.clickHere') + "]";
 						}
-							
+						
 						this.id('tagsLabel').value = Zotero.getString('pane.item.tags') + " " + v;
 					]]>
 				</body>
@@ -412,7 +420,15 @@
 					<![CDATA[
 						var relatedList = this.item.relatedItemsBidirectional;
 						if (relatedList.length > 0) {
-							this.id('seeAlsoPopup').showPopup(this.id('seeAlsoLabel'),-1,-1,'popup',0,0);
+							if (this.id('seeAlsoPopup').openPopupAtScreen) {
+								var x = this.boxObject.screenX;
+								var y = this.boxObject.screenY;
+								this.id('seeAlsoPopup').openPopupAtScreen(x, y, false);
+							}
+							// Deprecated method for Firefox 2
+							else {
+								this.id('seeAlsoPopup').showPopup(this.id('seeAlsoLabel'), -1, -1, 'popup');
+							}
 						}
 						else {
 							this.id('seeAlso').add();