From b8f7ce7ecb5ad068344f1b1c8fdac39b42e3033d Mon Sep 17 00:00:00 2001
From: Dan Stillman <dstillman@zotero.org>
Date: Mon, 2 Apr 2007 18:35:53 +0000
Subject: [PATCH] Fixes #602, https://www.zotero.org/trac/ticket/602

I just set a min-height to not collapse the top part when it's empty. Could add the parent item type in brackets, but I don't know that it's necessary.
---
 chrome/content/zotero/bindings/noteeditor.xml      | 2 +-
 chrome/skin/default/zotero/bindings/noteeditor.css | 6 ++++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/chrome/content/zotero/bindings/noteeditor.xml b/chrome/content/zotero/bindings/noteeditor.xml
index 0d857c263..528187ebd 100644
--- a/chrome/content/zotero/bindings/noteeditor.xml
+++ b/chrome/content/zotero/bindings/noteeditor.xml
@@ -190,7 +190,7 @@
 		
 		<content>
 			<xul:vbox xbl:inherits="flex">
-				<xul:label id="citeLabel" onclick="this.parentNode.parentNode.selectParent()"/>
+				<xul:label id="citeLabel" onclick="document.getBindingParent(this).selectParent()"/>
 				<xul:textbox id="noteField" multiline="true" type="timed" timeout="1000" flex="1" onkeydown="document.getBindingParent(this).handleKeyDown(event)" oncommand="document.getBindingParent(this).save();"/>
 				<xul:hbox>
 					<xul:linksbox id="links" flex="1"/>
diff --git a/chrome/skin/default/zotero/bindings/noteeditor.css b/chrome/skin/default/zotero/bindings/noteeditor.css
index 6031da38c..de25ae5f4 100644
--- a/chrome/skin/default/zotero/bindings/noteeditor.css
+++ b/chrome/skin/default/zotero/bindings/noteeditor.css
@@ -1,3 +1,9 @@
+/* Don't collapse blank note parent labels, since it prevents access to parent */
+#citeLabel
+{
+	min-height: 1.25em;
+}
+
 #citeLabel:hover
 {
 	cursor: pointer !important;