WoWInterface - Help with Devloping a "One Bag Addon"

WoWInterface (https://www.wowinterface.com/forums/index.php)

- Lua/XML Help (https://www.wowinterface.com/forums/forumdisplay.php?f=16)

- - Help with Devloping a "One Bag Addon" (https://www.wowinterface.com/forums/showthread.php?t=54944)

Help with Devloping a "One Bag Addon"

I've been trying to make a "One Bag Addon" for years that is lightweight and not too complicated to make, but my knowledge isn't the biggest, and i don't really know how get started for this.

So if anyone can help me out how to get started on this i will be very thankful!

I'd start with a list of everything you want the addon to do. Then figure out what events you want the addon to respond to in order to fulfill those needs, and work forward from that. When building the UI, you might want to consider design aspects like how many rows or columns do you want buttons to show up in and anything else you want shown. Planning out these things before you start coding is a major step to the development of an addon.

Not particularly helpful, but just a suggestion for a single feature that I think is important. If you have an option to hide empty bag slots (you should), it would be a good idea to always show a single empty slot. That will be handful when you want to split stacks. A text indicanting how many empty slots are left is enough.

Arkinventory, the best bag addon I know, doesn't have this feature despite having around 30k lines of code which is kinda of funny.

Take a look at Backpack, while by default it's categorized, you can disable all the categories and you'll get a all-in-one solution (it also has that empty slot feature that Banknorris mentioned).

It's also designed as a framework, allowing you to create your own layout (and distribute it), in some way it could be considered a successor to cargBags.

I know exactly how i want it to be, I want it to have 12 buttons per row and the maxium of coloums.

I already have my "layout/skin" figure.

How it currently looks like (Nothing super fancy just simple)

I want emtpy slots to always be shown. (But can probly work on some feature to hide it later, not so important right now just wanna get this "one bag feature" first)

P3lim: I have checked it out, but i just want to build something myself. And it would take a lot of work to rebuild yours into my UI :)

You've been trying for years and don't know how to get started?
You can start by looking at https://github.com/tekkub/wow-ui-sou...ainerFrame.lua
or another inventory addons, but they could be more complicated

And still, i would just reskin Bagnon, and it's probably quite lightweight without BagBrother

Quote:

Originally Posted by d87 (Post 321429)

Looking at that won't get me anywhere really, i need help with building a "one bag feature" and i dont really know how do it or start or which functions i need.

Bagnon is just too big and too many features, libs, etc. But thanks anyway.

https://github.com/tekkub/wow-ui-sou...ainerFrame.xml
https://github.com/tekkub/wow-ui-sou...ainerFrame.lua

As d87 pointed out, these ARE exactly where you should start STUDYING. This is how Blizzard themselves make the bags. ALL the functions in the lua will be needed to simply maintain the existing features. The .xml will also be needed unless you plan to convert it all manually into .lua code.

Quote:

Originally Posted by tonyis3l33t (Post 321433)

I understand, what you mean.

Anyway here is something i scrapped up.. Doesn't work really how i want it to be, i just want it be parented to ContainerFrame1 (Backpack). I want the "one bag" to act without having to use all the togglebags etc etc. :)

Lua Code:

local function CheckSlots()
    local numBags = 1
    for i = 1, NUM_BAG_FRAMES do
        local bagName = "ContainerFrame"..i+1
        if _G[bagName]:IsShown() and not _G[bagName.."BackgroundTop"]:GetTexture():find("Bank") then
            numBags = numBags + 1
        end
    end
    return numBags
end
 
local Spacing = 4
local bu, con, bag, col, row
local buttons, bankbuttons = {}, {}
 
local MoveButtons = function(table, frame, extraHeight)
    local columns = ceil(sqrt(#table))
    local iconSize = 32
 
    col, row = 0, 0
    for i = 1, #table do
        bu = table[i]
        bu:ClearAllPoints()
        bu:SetPoint("TOPLEFT", frame, "TOPLEFT", col * (iconSize + Spacing) + 3, -1 * row * (iconSize + Spacing) - 3)
        if(col > (columns - 2)) then
            col = 0
            row = row + 1
        else
            col = col + 1
        end
    end
 
    frame:SetHeight((row + (col==0 and 0 or 1)) * (iconSize + Spacing) + 19 + (extraHeight or 0))
    frame:SetWidth(columns * iconSize + Spacing * (columns - 1) + 6)
    col, row = 0, 0
end
 
local BagHolder = CreateFrame("Button", "BagsHolder", UIParent)
BagHolder:SetPoint("BOTTOMRIGHT", UIParent, "BOTTOMRIGHT", -30, 30)
BagHolder:SetFrameStrata("HIGH")
BagHolder:Hide()
 
local ReanchorButtons = function()
    table.wipe(buttons)
    for f = 1, CheckSlots() do
        con = "ContainerFrame"..f
        
        for i = GetContainerNumSlots(_G[con]:GetID()), 1, -1 do
            bu = _G[con.."Item"..i]
            tinsert(buttons, bu)
        end
    end
 
    MoveButtons(buttons, BagHolder)
 
    BagHolder:Show()
end
 
local CloseBags = function()
    BagHolder:Hide()
    for i = 0, 11 do
        CloseBag(i)
    end
end
 
local CloseBags2 = function()
    BagHolder:Hide()
end
 
local OpenBags = function()
    for i = 0, 4 do
        OpenBag(i)
    end
end
 
local ToggleBags = function()
    if(IsBagOpen(0)) then
        CloseBags()
    else
        OpenBags()
    end
end
 
for i = 1, 5 do
    local bag = _G["ContainerFrame"..i]
    hooksecurefunc(bag, "Show", ReanchorButtons)
    hooksecurefunc(bag, "Hide", CloseBags2)
end
 
ToggleBackpack = ToggleBags
ToggleBag = ToggleBags
OpenAllBags = OpenBags
OpenBackpack = OpenBags
CloseAllBags = CloseBags

One major thing to keep in mind is some functions like UseContainerItem() are protected functions. Blizzard can use them freely, but we have jump through hoops to implement the same functionality.