From ed0894f16dbc99cd80e3d5708cc81d3ec9785658 Mon Sep 17 00:00:00 2001
From: "vjekoslav.krenek@gmail.com"
 <313787825+svart2521@users.noreply.github.com>
Date: Tue, 15 Sep 2026 10:06:12 +0200
Subject: [PATCH] Fix: Quest Tracker Visibility Bug

Bug:

Issue: The shared "Instances" Show/Hide visibility axis (used by Quest
Tracker and every other module offering "Hide for Instances") never
matched battlegrounds. Reporter confirmed a random battleground stayed
visible with "Hide for Instances" checked, while dungeons/raids/delves
correctly hid.
Fix: EllesmereUI.IsInInstancedContent() early-returned false whenever
GetInstanceInfo()'s difficultyID was <= 0, before ever checking
instanceType. Battlegrounds report a difficultyID of 0 (PvP content has
no difficulty setting), so that guard tripped for every battleground
regardless of the instanceType == "pvp" check further down. Removed the
redundant difficultyID gate; the instanceType whitelist already
correctly scopes the probe to real instanced content.
---
 EllesmereUI.lua | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/EllesmereUI.lua b/EllesmereUI.lua
index 866cf8e06..52b8ee067 100644
--- a/EllesmereUI.lua
+++ b/EllesmereUI.lua
@@ -12247,9 +12247,7 @@ local DRUID_MOUNT_FORM_SPELLS = {
 -- "any" match verdict. A garrison reports a difficulty but is not instanced content
 -- for this axis, which is why the difficulty test alone is not enough.
 function EllesmereUI.IsInInstancedContent()
-    local _, iType, diffID = GetInstanceInfo()
-    diffID = tonumber(diffID) or 0
-    if diffID <= 0 then return false end
+    local _, iType = GetInstanceInfo()
     if C_Garrison and C_Garrison.IsOnGarrisonMap and C_Garrison.IsOnGarrisonMap() then
         return false
     end