Added a right-click menu option to permanently hide all loadouts from a specific character (closes #30)

Author: Numynum

core/Config.lua

@@ -33,6 +33,7 @@ function Config:Initialize()
         sideBarInactiveElementTextColor = { r = 1, g = 1, b = 1, a = 1 },
         sideBarInactiveElementBackgroundColor = { r = 0, g = 0, b = 0, a = 0.5 },
         sideBarInactiveElementHighlightBackgroundColor = { r = 0.5, g = 0.5, b = 0.5, a = 0.5 },
+        characterVisibility = {},
     };
     for key, value in pairs(self.defaultConfig) do
         if TLM.db.config[key] == nil then
@@ -116,6 +117,14 @@ You can create a leveling build yourself, either using the IcyVeins talent calcu
                 descStyle = "inline",
                 width = "full",
             },
+            resetCharacterVisibility = {
+                order = orderCount(),
+                type = "execute",
+                name = "Reset Hidden Characters",
+                desc = "Resets characters whose loadouts were hidden in the sidebar.",
+                func = function() self:ClearCharacterVisibility(); end,
+                width = "double",
+            },
             sideBarColors = {
                 order = orderCount(),
                 type = "description",
@@ -200,10 +209,13 @@ You can create a leveling build yourself, either using the IcyVeins talent calcu
         },
     };
 
-    return options
+    return options;
 end
 
-Config.Event = { OptionValueChanged = 'OptionValueChanged' };
+Config.Event = {
+    OptionValueChanged = 'OptionValueChanged',
+    CharacterVisibilityChanged = 'CharacterVisibilityChanged',
+};
 
 function Config:RegisterOptions()
     LibStub("AceConfig-3.0"):RegisterOptionsTable(addonName, self:GetOptions());
@@ -229,6 +241,20 @@ function Config:IsOptionDisabled(option)
     return false;
 end
 
+function Config:ClearCharacterVisibility()
+    TLM.db.config.characterVisibility = {};
+    self:TriggerEvent(self.Event.CharacterVisibilityChanged);
+end
+
+function Config:SetCharacterShown(characterName, isShown)
+    TLM.db.config.characterVisibility[characterName] = isShown;
+    self:TriggerEvent(self.Event.CharacterVisibilityChanged);
+end
+
+function Config:IsCharacterShown(characterName)
+    return TLM.db.config.characterVisibility[characterName] ~= false;
+end
+
 --- @param option TLM_ConfigOptions
 function Config:GetConfig(option, default)
     local value = not self:IsOptionDisabled(option) and TLM.db.config[option];

modules/SideBarMixin.lua

@@ -199,6 +199,7 @@ function SideBarMixin:OnDisable()
     self:UnhookAll();
 
     API:UnregisterCallback(API.Event.LoadoutListUpdated, self);
+    Config:UnregisterCallback(Config.Event.CharacterVisibilityChanged, self);
 end
 
 function SideBarMixin:SetupHook()
@@ -216,6 +217,7 @@ function SideBarMixin:SetupHook()
     self:SecureHook(talentsTab, "OnUpdate", "OnTalentsChanged");
 
     API:RegisterCallback(API.Event.LoadoutListUpdated, self.RefreshSideBarData, self);
+    Config:RegisterCallback(Config.Event.CharacterVisibilityChanged, self.RefreshSideBarData, self);
 end
 
 do
@@ -850,6 +852,10 @@ function SideBarMixin:GenerateMenu(rootDescription, frame, loadoutInfo)
         rootDescription:CreateButton("Remove all loadouts from this character from the list", function()
             StaticPopup_Show(self.removeFromListBulkDialogName, loadoutInfo.owner, nil, loadoutInfo);
         end);
+        rootDescription:CreateButton("Permanently hide all loadouts from this character", function()
+            Config:SetCharacterShown(loadoutInfo.owner, false);
+            TLM:Printf("Loadouts from %s are now hidden. You can reset this in the config.", loadoutInfo.owner);
+        end);
     end
 end
 
@@ -1055,21 +1061,23 @@ function SideBarMixin:RefreshSideBarData()
     local activeLoadoutID = self.activeLoadout and self.activeLoadout.id or nil;
     local dataProviderEntries = {}
     for _, loadout in pairs(loadouts) do
-        --- @type TLM_SideBarLoadoutInfo
-        local loadout = loadout ---@diagnostic disable-line: assign-type-mismatch, redefined-local
-        --- @type number?
-        local parentID = loadout.parentMapping and loadout.parentMapping[0];
-        loadout.parentID = parentID;
-        --- @type TLM_SideBarDataProviderEntry
-        local entry = {
-            text = loadout.displayName,
-            data = loadout,
-            isActive = loadout.id == activeLoadoutID,
-            parentID = parentID,
-        };
-        table.insert(dataProviderEntries, entry);
-        if loadout.id == activeLoadoutID then
-            foundActiveLoadout = true;
+        if loadout.playerIsOwner or not loadout.owner or Config:IsCharacterShown(loadout.owner) then
+            --- @type TLM_SideBarLoadoutInfo
+            local loadout = loadout ---@diagnostic disable-line: assign-type-mismatch, redefined-local
+            --- @type number?
+            local parentID = loadout.parentMapping and loadout.parentMapping[0];
+            loadout.parentID = parentID;
+            --- @type TLM_SideBarDataProviderEntry
+            local entry = {
+                text = loadout.displayName,
+                data = loadout,
+                isActive = loadout.id == activeLoadoutID,
+                parentID = parentID,
+            };
+            table.insert(dataProviderEntries, entry);
+            if loadout.id == activeLoadoutID then
+                foundActiveLoadout = true;
+            end
         end
     end
     if not foundActiveLoadout then