From 0df7f4afd4489f38ddfcb10f92f4c75bf87d011e Mon Sep 17 00:00:00 2001
From: Dan Stillman <dstillman@zotero.org>
Date: Tue, 17 Mar 2009 07:59:41 +0000
Subject: [PATCH] Always show "Reindex item", even if item is indexed, and
 flicker the refresh icon briefly on click to show that it's doing something

---
 chrome/content/zotero/bindings/attachmentbox.xml | 2 +-
 chrome/content/zotero/xpcom/fulltext.js          | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/chrome/content/zotero/bindings/attachmentbox.xml b/chrome/content/zotero/bindings/attachmentbox.xml
index 4e881c049..5eb42f95d 100644
--- a/chrome/content/zotero/bindings/attachmentbox.xml
+++ b/chrome/content/zotero/bindings/attachmentbox.xml
@@ -508,7 +508,7 @@
 				
 				<hbox id="index-box">
 					<label id="index-status"/>
-					<toolbarbutton id="reindex" oncommand="ZoteroPane.reindexItem()"/>
+					<toolbarbutton id="reindex" oncommand="this.hidden = true; setTimeout(function () { ZoteroPane.reindexItem(); }, 50)"/>
 				</hbox>
 				
 				<zoteronoteeditor id="note-editor" notitle="1" flex="1"/>
diff --git a/chrome/content/zotero/xpcom/fulltext.js b/chrome/content/zotero/xpcom/fulltext.js
index a01a8b62d..fd88aa622 100644
--- a/chrome/content/zotero/xpcom/fulltext.js
+++ b/chrome/content/zotero/xpcom/fulltext.js
@@ -830,6 +830,9 @@ Zotero.Fulltext = new function(){
 				case this.INDEX_STATE_UNAVAILABLE:
 				case this.INDEX_STATE_UNINDEXED:
 				case this.INDEX_STATE_PARTIAL:
+				
+				// TODO: automatically reindex already-indexed attachments?
+				case this.INDEX_STATE_INDEXED:
 					return true;
 			}
 		}