From 4da6bba5659d3db3d54a84d68c8f5b3e1c29ec9e Mon Sep 17 00:00:00 2001 From: Salikh Gurgenidze Date: Thu, 25 Dec 2025 01:54:28 +0400 Subject: [PATCH] feat: reverse stack sorting --- Sorting/SortEngine.lua | 30 ++++++++++++++++++++++++++++-- UI/SettingsPopup.lua | 40 ++++++++++++++++++++++++++++++++++++++++ UI/SettingsPopup.xml | 33 ++++++++++++++++++++++++++------- 3 files changed, 94 insertions(+), 9 deletions(-) diff --git a/Sorting/SortEngine.lua b/Sorting/SortEngine.lua index 911f617..fd5b90d 100644 --- a/Sorting/SortEngine.lua +++ b/Sorting/SortEngine.lua @@ -489,6 +489,9 @@ local function AddSortKeys(items) item.texturePattern = "" item.invertedCount = 0 item.invertedItemID = 0 + item.maxStackCount = 1 + item.isStackable = false + item.stackCount = 1 else -- Check if item is equippable (Armor or Weapon category) local isEquippable = itemCategory == "Armor" or itemCategory == "Weapon" @@ -576,6 +579,11 @@ local function AddSortKeys(items) item.invertedCount = -(tonumber(item.data.count) or 1) item.invertedItemID = -tonumber(itemID) + -- Stack info for reverse stack sorting + item.maxStackCount = tonumber(itemStackCount) or 1 + item.isStackable = item.maxStackCount > 1 + item.stackCount = tonumber(item.data.count) or 1 + -- Name for alphabetical sorting item.itemName = item.name or "" end @@ -586,6 +594,15 @@ end local function SortItems(items) AddSortKeys(items) + -- Get reverse stack sort setting from DB + local reverseStackSort = false + if addon.Modules.DB and addon.Modules.DB.GetSetting then + reverseStackSort = addon.Modules.DB:GetSetting("reverseStackSort") + if reverseStackSort == nil then + reverseStackSort = false + end + end + table.sort(items, function(a, b) -- 1. Priority items first (Hearthstone, etc.) if a.priority ~= b.priority then @@ -677,9 +694,18 @@ local function SortItems(items) if a.invertedItemLevel ~= b.invertedItemLevel then return a.invertedItemLevel < b.invertedItemLevel end - -- Then by stack count (larger stacks first) + -- Then by stack count + -- If reverse stack sort is enabled AND both items are the same stackable item, + -- place smaller stacks before larger ones if a.invertedCount ~= b.invertedCount then - return a.invertedCount < b.invertedCount + -- Check if both items are the same stackable item (same itemID) + if reverseStackSort and a.isStackable and b.isStackable and a.invertedItemID == b.invertedItemID then + -- Reverse: smaller stacks first (compare stackCount ascending) + return a.stackCount < b.stackCount + else + -- Normal: larger stacks first (use invertedCount) + return a.invertedCount < b.invertedCount + end end -- Final stable sort: preserve original collection order for identical items -- This prevents unnecessary reshuffling when items are already sorted diff --git a/UI/SettingsPopup.lua b/UI/SettingsPopup.lua index 076d209..56f4030 100644 --- a/UI/SettingsPopup.lua +++ b/UI/SettingsPopup.lua @@ -973,6 +973,46 @@ function Guda_SettingsPopup_BankViewTypeButton_OnClick() end end +-- Reverse Stack Sort Checkbox OnLoad +function Guda_SettingsPopup_ReverseStackSortCheckbox_OnLoad(self) + local text = getglobal(self:GetName().."Text") + if text then + text:SetText("Reverse Stack Sort") + + -- Increase font size + local font, _, flags = text:GetFont() + if font then + text:SetFont(font, 13, flags) + end + end + + -- Tooltip + self.tooltipText = "When enabled, smaller stacks of the same item will be sorted before larger stacks (e.g., stack of 16 before stack of 20)." + + local reverseStackSort = false + if Guda and Guda.Modules and Guda.Modules.DB then + reverseStackSort = Guda.Modules.DB:GetSetting("reverseStackSort") + if reverseStackSort == nil then + reverseStackSort = false + end + end + + self:SetChecked(reverseStackSort and 1 or 0) +end + +-- Reverse Stack Sort Checkbox OnClick +function Guda_SettingsPopup_ReverseStackSortCheckbox_OnClick(self) + local isChecked = self:GetChecked() == 1 + + -- Save setting + if Guda and Guda.Modules and Guda.Modules.DB then + Guda.Modules.DB:SetSetting("reverseStackSort", isChecked) + end + + -- Note: Sorting will use the new setting on next sort operation + -- No immediate UI update needed +end + -- Initialize function SettingsPopup:Initialize() Guda:Debug("Settings popup initialized") diff --git a/UI/SettingsPopup.xml b/UI/SettingsPopup.xml index ae35e62..8763f71 100644 --- a/UI/SettingsPopup.xml +++ b/UI/SettingsPopup.xml @@ -395,12 +395,31 @@ - -