From 5484fdb01de6fd54b681cb7eff35220034cce1aa Mon Sep 17 00:00:00 2001
From: Dan Stillman <dstillman@zotero.org>
Date: Fri, 2 Dec 2011 19:31:28 -0500
Subject: [PATCH] Adjust some note editor ids for clarity when debugging

---
 chrome/content/zotero/bindings/attachmentbox.xml  | 4 ++--
 chrome/content/zotero/bindings/noteeditor.xml     | 2 +-
 chrome/content/zotero/bindings/styled-textbox.xml | 1 +
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/chrome/content/zotero/bindings/attachmentbox.xml b/chrome/content/zotero/bindings/attachmentbox.xml
index bff649662..a2cedddc7 100644
--- a/chrome/content/zotero/bindings/attachmentbox.xml
+++ b/chrome/content/zotero/bindings/attachmentbox.xml
@@ -297,7 +297,7 @@
 					}
 					
 					// Note editor
-					var noteEditor = this._id('note-editor');
+					var noteEditor = this._id('attachment-note-editor');
 					if (this.displayNote) {
 						if (this.displayNoteIfEmpty || this.item.getNote() != '') {
 							Zotero.debug("setting links on top");
@@ -542,7 +542,7 @@
 					</rows>
 				</grid>
 				
-				<zoteronoteeditor id="note-editor" notitle="1" flex="1"/>
+				<zoteronoteeditor id="attachment-note-editor" notitle="1" flex="1"/>
 				
 				<button id="select-button" hidden="true"/>
 			</vbox>
diff --git a/chrome/content/zotero/bindings/noteeditor.xml b/chrome/content/zotero/bindings/noteeditor.xml
index a0d8e64eb..6929f9fcd 100644
--- a/chrome/content/zotero/bindings/noteeditor.xml
+++ b/chrome/content/zotero/bindings/noteeditor.xml
@@ -497,7 +497,7 @@
 			</method>
 		</implementation>
 		<content>
-			<xul:vbox id="note-editor" xbl:inherits="flex">
+			<xul:vbox xbl:inherits="flex">
 				<xul:grid>
 					<xul:columns>
 						<xul:column/>
diff --git a/chrome/content/zotero/bindings/styled-textbox.xml b/chrome/content/zotero/bindings/styled-textbox.xml
index 35d564b20..5c75532ad 100644
--- a/chrome/content/zotero/bindings/styled-textbox.xml
+++ b/chrome/content/zotero/bindings/styled-textbox.xml
@@ -103,6 +103,7 @@
 
 				this._constructed = true;
 				
+				// Don't load if a value hasn't yet been set
 				if (this._loadOnConstruct) {
 					this._load();
 				}