Mercurial > wow > worldplan

diff QuestPOI.lua @ 73:25f7dbc20a72

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
fix WorldMapTooltip getting stranded when flightmap is closed - quests that are not valid spell targets will have their icon go red - auto-complete scrolls will no longer hide pin frames; instead they become click through and blizzard POIs are activated beneath for taint-free interaction
author Nenue
date Fri, 07 Apr 2017 07:20:28 -0400
parents 6db0b9696936
children 2ba11b25aa7f
line wrap: on
line diff
--- a/QuestPOI.lua	Fri Apr 07 01:07:48 2017 -0400
+++ b/QuestPOI.lua	Fri Apr 07 07:20:28 2017 -0400
@@ -40,8 +40,8 @@
 local dprint = DEVIAN_WORKSPACE and function(...) _G.print('WQData', ...) end or nop
 local QuestPOI = WorldPlanPOIMixin
 
-local pinBaseIndex = 1300
-local overlayBaseIndex = 1350
+local pinBaseIndex = 1500
+local overlayBaseIndex = 1580
 local previousHighlight
 
 local DATA_DEBUG = false
@@ -912,6 +912,18 @@
   self:UpdateStatus()
   self.isStale = nil
 
+
+  if SpellCanTargetQuest() then
+    if IsQuestIDValidSpellTarget(self.questID) then
+      icon:SetVertexColor(1,1,1)
+    else
+      icon:SetVertexColor(1,0,0,1)
+    end
+    self:EnableMouse(false)
+  else
+    self:EnableMouse(true)
+  end
+
   -- signal filter info update
   WorldPlanSummary.isStale = true
 end
@@ -931,16 +943,6 @@
   local filtered
   local canShow = TQ_IsActive(self.questID)
 
-  if db.canTargetQuests then
-    self:EnableMouse(false)
-    if not IsQuestIDValidSpellTarget(self.questID) then
-      self.hideReason = "Not a valid spell target."
-      self:SetShown(false)
-      return
-    end
-  else
-    self:EnableMouse(true)
-  end
 
   if qType == LE_QUEST_TAG_TYPE_PROFESSION then
     if not(self.isKnownProfession or db.Config.ShowAllProfessionQuests) then