fix(tooltip): remove misleading data-source attribution line
The Source: line cannot accurately reflect per-pin provenance without changing every spawn registration site. Until that infrastructure exists, the label is misleading — showing Questie DB even when data came from the learner, or Learner when it came from AscensionDB. Remove from both MapIconTooltip (world-map pins) and Tooltip (unit/object hover).
This commit is contained in:
@@ -70,11 +70,7 @@ local DEFAULT_WAYPOINT_HOVER_COLOR = { 0.93, 0.46, 0.13, 0.8 }
|
|||||||
local lastTooltipShowTimestamp = GetTime()
|
local lastTooltipShowTimestamp = GetTime()
|
||||||
|
|
||||||
local function _GetWorldMapTooltipSourceLine()
|
local function _GetWorldMapTooltipSourceLine()
|
||||||
local parts = { "Questie DB" }
|
return nil
|
||||||
if QuestieComms and (QuestieComms.remotePlayerEnabled or QuestieComms.data) then
|
|
||||||
tinsert(parts, "Comms")
|
|
||||||
end
|
|
||||||
return "|cFF808080Source: " .. table.concat(parts, " + ") .. "|r"
|
|
||||||
end
|
end
|
||||||
|
|
||||||
function MapIconTooltip:Show()
|
function MapIconTooltip:Show()
|
||||||
|
|||||||
@@ -116,44 +116,11 @@ local function _GetQuestObjectiveSummary(questId)
|
|||||||
end
|
end
|
||||||
|
|
||||||
local function _BuildTooltipSourceLine(sourceFlags)
|
local function _BuildTooltipSourceLine(sourceFlags)
|
||||||
if not sourceFlags then
|
|
||||||
return nil
|
return nil
|
||||||
end
|
end
|
||||||
|
|
||||||
local parts = {}
|
|
||||||
if sourceFlags.static then
|
|
||||||
tinsert(parts, "Static DB")
|
|
||||||
end
|
|
||||||
if sourceFlags.learner then
|
|
||||||
tinsert(parts, "Learner")
|
|
||||||
end
|
|
||||||
if sourceFlags.comms then
|
|
||||||
tinsert(parts, "Comms")
|
|
||||||
end
|
|
||||||
|
|
||||||
if table.getn(parts) == 0 then
|
|
||||||
return nil
|
|
||||||
end
|
|
||||||
|
|
||||||
return "|cFF808080Source: " .. table.concat(parts, " + ") .. "|r"
|
|
||||||
end
|
|
||||||
|
|
||||||
local function _GetTooltipSourceLine(key)
|
local function _GetTooltipSourceLine(key)
|
||||||
local sourceFlags = {}
|
return nil
|
||||||
|
|
||||||
local QuestieLearner = QuestieLoader:ImportModule("QuestieLearner")
|
|
||||||
local mode = QuestieLearner and QuestieLearner.GetDataSourceMode and QuestieLearner:GetDataSourceMode() or "auto"
|
|
||||||
if mode == "learner" then
|
|
||||||
sourceFlags.learner = true
|
|
||||||
else
|
|
||||||
sourceFlags.static = true
|
|
||||||
end
|
|
||||||
|
|
||||||
if QuestieComms and QuestieComms.data and QuestieComms.data.KeyExists and QuestieComms.data:KeyExists(key) then
|
|
||||||
sourceFlags.comms = true
|
|
||||||
end
|
|
||||||
|
|
||||||
return _BuildTooltipSourceLine(sourceFlags)
|
|
||||||
end
|
end
|
||||||
|
|
||||||
---@param questId number
|
---@param questId number
|
||||||
|
|||||||
Reference in New Issue
Block a user