From c9dbd34e010db25460694960b82cf48a47f6f56d Mon Sep 17 00:00:00 2001
From: Dan Stillman <dstillman@zotero.org>
Date: Tue, 17 Dec 2013 18:12:42 -0500
Subject: [PATCH] Fix issue with old-style persisted 'collapsed' attribute on
 tag selector

If localstore.rdf has the tag selector persisted closed from before
zotero-persist, the 'state' attribute on the splitter that we now use
won't cause the tag selector to open.
---
 chrome/content/zotero/zoteroPane.xul | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/chrome/content/zotero/zoteroPane.xul b/chrome/content/zotero/zoteroPane.xul
index 7d6acb5d7..1f2487649 100644
--- a/chrome/content/zotero/zoteroPane.xul
+++ b/chrome/content/zotero/zoteroPane.xul
@@ -319,7 +319,13 @@
 							zotero-persist="state">
 						<grippy oncommand="ZoteroPane_Local.toggleTagSelector()"/>
 					</splitter>
-					<zoterotagselector id="zotero-tag-selector" zotero-persist="height,showAutomatic,filterToScope"
+					<!-- 'collapsed' is no longer necessary here due to the persisted 'state' on
+					     zotero-tags-splitter, but without this an old-style entry for 'collapsed'
+					     in localstore.rdf can cause the pane to always be closed on Zotero open.
+					     
+					     TODO: deal with this some other way?
+					-->
+					<zoterotagselector id="zotero-tag-selector" zotero-persist="height,collapsed,showAutomatic,filterToScope"
 						oncommand="ZoteroPane_Local.updateTagFilter()"/>
 				</vbox>