Special item icons now disappear when their quest is complete.

Development/QHBuild.sh

@@ -112,7 +112,7 @@ do
   rm $lua.bak
 done
 
-zip -qdgds 1m -r QuestHelper-4.0.3a.${release_string}.zip QuestHelper/
+zip -qdgds 1m -r QuestHelper-4.0.6.${release_string}.zip QuestHelper/
 
 echo Moving the package up one directory level.
 mv -v *.zip ..

QuestHelper/tracker.lua

@@ -368,36 +368,45 @@ local function addItem(objective, y, meta)
   local spacer = 0
   -- hacky - progress only shows up if we're not on a metaobjective. wheee
   if objective.type_quest and objective.type_quest.index and not objective.progress and GetQuestLogSpecialItemInfo(objective.type_quest.index) then
-    item.specitem = table.remove(specitem_unused)
-    if not item.specitem then
-      item.specitem = CreateFrame("BUTTON", "QH_SpecItem_" .. tostring(specitem_max), item, "WatchFrameItemButtonTemplate")
-      QuestHelper: Assert(item.specitem)
+    if not objective.type_quest.done then
+      item.specitem = table.remove(specitem_unused)
+      if not item.specitem then
+        item.specitem = CreateFrame("BUTTON", "QH_SpecItem_" .. tostring(specitem_max), item, "WatchFrameItemButtonTemplate")
+        QuestHelper: Assert(item.specitem)
       
-      local rangey = _G["QH_SpecItem_" .. tostring(specitem_max) .. "HotKey"]
-      QuestHelper: Assert(rangey)
-      local fn, fh, ff = rangey:GetFont()
-      rangey:SetFont("Fonts\\ARIALN.TTF", fh, ff)
-      rangey:SetText(RANGE_INDICATOR)
-      rangey:ClearAllPoints()
-      rangey:SetPoint("BOTTOMRIGHT", item.specitem, "BOTTOMRIGHT", 0, 2)
+        local rangey = _G["QH_SpecItem_" .. tostring(specitem_max) .. "HotKey"]
+        QuestHelper: Assert(rangey)
+        local fn, fh, ff = rangey:GetFont()
+        rangey:SetFont("Fonts\\ARIALN.TTF", fh, ff)
+        rangey:SetText(RANGE_INDICATOR)
+        rangey:ClearAllPoints()
+        rangey:SetPoint("BOTTOMRIGHT", item.specitem, "BOTTOMRIGHT", 0, 2)
       
-      specitem_max = specitem_max + 1
+        specitem_max = specitem_max + 1
+      end
+    
+      item.specitem:SetScale(0.9)
+      item.specitem:ClearAllPoints()
+      item.specitem:SetParent(item)
+      item.specitem:SetPoint("TOPRIGHT", item, "TOPLEFT", 0, 0)
+    
+      local _, tex, charges = GetQuestLogSpecialItemInfo(objective.type_quest.index)
+      item.specitem:SetID(objective.type_quest.index)
+      SetItemButtonTexture(item.specitem, tex)
+      item.specitem.rangeTimer = -1 -- This makes the little dot go away. Why does it do that?
+      item.specitem.charges = charges
+    
+      item.specitem:Show()
+    
+      spacer = h
+    else
+      -- Make the item go away.
+      if item.specitem then
+        item.specitem:Hide()
+        table.insert(specitem_unused, item.specitem)
+        item.specitem = nil
+      end
     end
-    
-    item.specitem:SetScale(0.9)
-    item.specitem:ClearAllPoints()
-    item.specitem:SetParent(item)
-    item.specitem:SetPoint("TOPRIGHT", item, "TOPLEFT", 0, 0)
-    
-    local _, tex, charges = GetQuestLogSpecialItemInfo(objective.type_quest.index)
-    item.specitem:SetID(objective.type_quest.index)
-    SetItemButtonTexture(item.specitem, tex)
-    item.specitem.rangeTimer = -1 -- This makes the little dot go away. Why does it do that?
-    item.specitem.charges = charges
-    
-    item.specitem:Show()
-    
-    spacer = h
   end
   
   return w+x+4, y+h, y+h+spacer