Skip to content

Commit

Permalink
Vignette scanning: option to alert on zone-wide vignettes
Browse files Browse the repository at this point in the history
I originally disabled these because it can be really spammy in zones
like Timeless Isles / Zereth Mortis, but with an option for people who
can't stand it I'll turn them back on by default.
  • Loading branch information
kemayo committed May 25, 2024
1 parent af8cb84 commit 0d70a1e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion scan/vignettes.lua
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ function module:OnInitialize()
enabled = true,
pointsofinterest = true,
visibleOnly = false,
zoneInfinite = true,
ignore = {
-- [id] = "name",
},
Expand Down Expand Up @@ -54,6 +55,7 @@ function module:OnInitialize()
args = {
enabled = config.toggle("Enabled", "Scan minimap vignettes (it's what Blizzard calls them, okay?)", 10),
pointsofinterest = config.toggle("World points-of-interest", "Show alerts for point of interest vignettes added to world map itself", 20),
zoneInfinite = config.toggle("Infinite distance vignettes", "Show alerts for vignettes that can be seen from across the entire zone (this can get really spammy in some zones)", 25),
visibleOnly = config.toggle("Wait until visible", "Don't notify until the vignette is actually visible on the minimap", 30),
ignore = {
type="group",
Expand Down Expand Up @@ -155,7 +157,7 @@ local function shouldShowNotVisible(vignetteInfo, zone)
-- it results in bursts of alerts when zoning into the Shadowlands area with the daily chests
return not module.db.profile.visibleOnly
end
if vignetteInfo.zoneInfiniteAOI then
if vignetteInfo.zoneInfiniteAOI and not module.db.profile.zoneInfinite then
-- It can be semi-seen from the entire zone, and so we should wait until it's actually-visible
return false
end
Expand Down

0 comments on commit 0d70a1e

Please sign in to comment.