From 59913cc5c02dc29b39644895d99d539ab75ee4ff Mon Sep 17 00:00:00 2001
From: Simon Kornblith <simon@simonster.com>
Date: Mon, 30 Jul 2012 03:08:38 -0400
Subject: [PATCH] Tweak notes/tags/related pane appearance

---
 chrome/content/zotero/bindings/relatedbox.xml      |  4 +++-
 chrome/content/zotero/bindings/tagsbox.xml         |  9 +++++++--
 chrome/content/zotero/itemPane.js                  |  2 ++
 chrome/content/zotero/itemPane.xul                 |  2 +-
 .../skin/default/zotero/bindings/attachmentbox.css |  2 +-
 chrome/skin/default/zotero/overlay.css             | 14 ++++++++++++++
 6 files changed, 28 insertions(+), 5 deletions(-)

diff --git a/chrome/content/zotero/bindings/relatedbox.xml b/chrome/content/zotero/bindings/relatedbox.xml
index cc2576c800..975c460666 100644
--- a/chrome/content/zotero/bindings/relatedbox.xml
+++ b/chrome/content/zotero/bindings/relatedbox.xml
@@ -107,6 +107,7 @@
 								related = Zotero.Items.get(related);
 								for (var i = 0; i < related.length; i++) {
 									var icon= document.createElement("image");
+									icon.className = "zotero-box-icon";
 									var type = Zotero.ItemTypes.getName(related[i].itemTypeID);
 									if (type=='attachment')
 									{
@@ -132,6 +133,7 @@
 									icon.setAttribute('src','chrome://zotero/skin/treeitem-' + type + '.png');
 							
 									var label = document.createElement("label");
+									label.className = "zotero-box-label";
 									label.setAttribute('value', related[i].getDisplayTitle());
 									label.setAttribute('crop','end');
 									label.setAttribute('flex','1');
@@ -284,7 +286,7 @@
 			</method>
 		</implementation>
 		<content>
-			<xul:vbox xbl:inherits="flex">
+			<xul:vbox xbl:inherits="flex" class="zotero-box">
 				<xul:hbox align="center">
 					<xul:label id="seeAlsoNum"/>
 					<xul:button id="addButton" label="&zotero.item.add;" oncommand="this.parentNode.parentNode.parentNode.add();" hidden="true"/>
diff --git a/chrome/content/zotero/bindings/tagsbox.xml b/chrome/content/zotero/bindings/tagsbox.xml
index c08ae15822..45f7677fd3 100644
--- a/chrome/content/zotero/bindings/tagsbox.xml
+++ b/chrome/content/zotero/bindings/tagsbox.xml
@@ -30,6 +30,10 @@
 			xmlns:xbl="http://www.mozilla.org/xbl"
 			xmlns:xul="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
 	<binding id="tags-box">
+		<resources>
+			<stylesheet src="chrome://zotero/skin/bindings/tagsbox.css"/>
+		</resources>
+		
 		<implementation>
 			<field name="clickHandler"/>
 			<field name="_tagColors"/>
@@ -169,6 +173,7 @@
 						}
 						
 						var icon = document.createElement("image");
+						icon.className = "zotero-box-icon";
 						var iconFile = 'tag';
 						if (type == 0) {
 							icon.setAttribute('tooltiptext', Zotero.getString('pane.item.tags.icon.user'));
@@ -183,6 +188,7 @@
 						icon.setAttribute('onclick','if (this.nextSibling.inputField){ this.nextSibling.inputField.blur() }');
 						
 						var label = this.createValueElement(name, tabindex);
+						label.className = "zotero-box-label";
 						
 						if (this.editable) {
 							var remove = document.createElement("label");
@@ -759,12 +765,11 @@
 			</method>
 		</implementation>
 		<content>
-			<xul:scrollbox xbl:inherits="flex" orient="vertical" style="overflow:auto">
+			<xul:scrollbox xbl:inherits="flex" orient="vertical" style="overflow:auto" class="zotero-box">
 				<xul:hbox align="center">
 					<xul:label id="tagsNum"/>
 					<xul:button id="addButton" label="&zotero.item.add;" oncommand="document.getBindingParent(this).new();" hidden="true"/>
 				</xul:hbox>
-				<xul:separator class="thin"/>
 				<xul:grid>
 					<xul:columns>
 						<xul:column/>
diff --git a/chrome/content/zotero/itemPane.js b/chrome/content/zotero/itemPane.js
index 53bdee6433..0d161d0c4e 100644
--- a/chrome/content/zotero/itemPane.js
+++ b/chrome/content/zotero/itemPane.js
@@ -101,9 +101,11 @@ var ZoteroItemPane = new function() {
 					let id = notes[i].id;
 					
 					var icon = document.createElement('image');
+					icon.className = "zotero-box-icon";
 					icon.setAttribute('src','chrome://zotero/skin/treeitem-note.png');
 					
 					var label = document.createElement('label');
+					label.className = "zotero-box-label";
 					var title = Zotero.Notes.noteToTitle(notes[i].getNote());
 					title = title ? title : Zotero.getString('pane.item.notes.untitled');
 					label.setAttribute('value', title);
diff --git a/chrome/content/zotero/itemPane.xul b/chrome/content/zotero/itemPane.xul
index 9ccef42066..b1c0efdf0c 100644
--- a/chrome/content/zotero/itemPane.xul
+++ b/chrome/content/zotero/itemPane.xul
@@ -63,7 +63,7 @@
 						</tabpanel>
 						
 						<tabpanel flex="1" orient="vertical">
-							<vbox flex="1">
+							<vbox flex="1" class="zotero-box">
 								<hbox align="center">
 									<label id="zotero-editpane-notes-label"/>
 									<button id="zotero-editpane-notes-add" label="&zotero.item.add;" oncommand="ZoteroItemPane.addNote(event.shiftKey);"/>
diff --git a/chrome/skin/default/zotero/bindings/attachmentbox.css b/chrome/skin/default/zotero/bindings/attachmentbox.css
index 0c5b44fcfc..813816117a 100644
--- a/chrome/skin/default/zotero/bindings/attachmentbox.css
+++ b/chrome/skin/default/zotero/bindings/attachmentbox.css
@@ -72,7 +72,7 @@ row > label:first-child
 {
 	text-align: right;
 	font-weight: bold;
-	-moz-margin-start: 5px !important;
+	-moz-margin-start: 3px !important;
 	-moz-margin-end: 0 !important;
 	width: 62px;
 	text-align: right;
diff --git a/chrome/skin/default/zotero/overlay.css b/chrome/skin/default/zotero/overlay.css
index 28ea188af7..47bb891813 100644
--- a/chrome/skin/default/zotero/overlay.css
+++ b/chrome/skin/default/zotero/overlay.css
@@ -502,4 +502,18 @@
 {
 	visibility: visible !important;
 	display: inherit !important;
+}
+
+
+.zotero-box {
+	margin-left: 5px;
+}
+
+.zotero-box-icon {
+	width: 16px;
+	height: 16px;
+}
+
+.zotero-box-label {
+	margin-left: 3px !important;
 }
\ No newline at end of file