Beautycase borders on Grid2 units

[lerb]

So, I've been trying to find where Grid2 handles its healthbars so I can add !beautycase borders to them with its "createborderlight" function, but I can't find it. Anyone that can help me with this?

Or, if there's another way to skin the borders of the actual unitframes, not just the background frame, that would be great

Thanks in advance!

[lerb]

I've been trying to find something similar with "statusbar", or a createframe that creates the health statusbar, or just the background of each unit. Without luck, though.

[Phanx]

Here is a standalone version of the code I use to apply borders to Grid. It's been a while since I looked at the internals of Grid2, but I don't think the basics of frame setup is that different. Maybe it will give you some clues, especially the parts I highlighted in gold. The parts highlighted in green are specific to my border code, so may need changes for use with Beautycase.

local function SkinGrid()
	local GridFrame = Grid and Grid:GetModule("GridFrame")
	if GridFrame and GridFrame.registeredFrames then
		-- print("Adding borders to Grid frames")

		local function Grid_SetBackdropBorderColor(f, r, g, b, a)
			if a and a == 0 then
				f:SetBorderColor()
			else
				f:SetBorderColor(r, g, b)
			end
		end

		local function Grid_AddBorder(f)
			if not f.SetBorderColor then
				f:SetBorderSize(0.1)
				AddBorder(f)
				f.SetBackdropBorderColor = Grid_SetBackdropBorderColor
				f.SetBorderSize = noop
			end
		end

		for frame in pairs(GridFrame.registeredFrames) do
			Grid_AddBorder(_G[frame])
		end

		local o = GridFrame.RegisterFrame
		GridFrame.RegisterFrame = function(self, f)
			o(self, f)
			Grid_AddBorder(f)
		end

		return true
	end
end

if not SkinGrid() then
	local f = CreateFrame("Frame")
	f:RegisterEvent("ADDON_LOADED")
	f:RegisterEvent("PLAYER_LOGIN")
	f:SetScript("OnEvent", function()
		if SkinGrid() then
			f:UnregisterAllEvents()
			f:SetScript("OnEvent", nil)
			SkinGrid = nil
		end
	end)
end

[lerb]

Thanks for sharing Phanx! Really nice of you, I think I took water over my head on this one

I've moved back to Grid because I like it better, and trying to get your code to work. To be honest, LUA is still an unfamiliar thing for me so I don't really know what to do with this code. Could I please ask for your help?

Here are the borders I'm trying to add btw;

[Phanx]

Originally Posted by lerb To be honest, LUA is still an unfamiliar thing for me so I don't really know what to do with this code. Could I please ask for your help?

Create a folder named "GridBorders" in your AddOns folder. Inside it, create a file named "addon.lua" and paste the code into it, and then create a basic TOC file named "GridBorders.toc" and paste this into it:

## Interface: 40300
addon.lua

Also, I edited my previous post to highlight the lines that may need to be changed for use with Beautycase instead of my personal border addon.

[lerb]

Hi again Phanx! Hope you don't mind me necroing this thread.

I've been playing around with this again lately and got it working a little bit, but I can't get it to skin all the units. I've used /fstack to find out the names of the frames for the actual units in grid, not the background frame, and tried to attach a beautycase border to them with the beautycase function, which looks like this when I'm using your code;

		local function Grid_AddBorder(f)
			CreateBorderChat(GridLayoutHeader1UnitButton1, LeUI.media.bordersize, LeUI.bordercolor, LeUI.bordercolor, LeUI.bordercolor, 2)
		end

As you can see, if I hover myself in grid the frame is named GridLayoutHeader1UnitButton1. There was a couple of different frames to choose from but this one worked. Problem is, sometimes there's up to 40 of these. Do you have any clue how I could get a border on every unit? I guess I could create 40 CreateBorder lines, but that doesn't seem to be the correct way to go.

The different frames appearing when hovering the first unit in my party/raid with /fstack is;

1. GridLayoutHeader1UnitButton1
2. GridLayoutHeader1
3. GridLayoutFrame

Here's the full code as it looks right now, I have no way if it is efficient as it is now but at least it's partially working;

local function SkinGrid()
	local GridFrame = Grid and Grid:GetModule("GridFrame")
	if GridFrame and GridFrame.registeredFrames then
		-- print("Adding borders to Grid frames")

		local function Grid_AddBorder(f)
			CreateBorderChat(GridLayoutHeader1UnitButton1, LeUI.media.bordersize, LeUI.bordercolor, LeUI.bordercolor, LeUI.bordercolor, 2)
		end

		local o = GridFrame.RegisterFrame
		GridFrame.RegisterFrame = function(self, f)
			o(self, f)
			Grid_AddBorder(f)
		end

		return true
	end
end

if not SkinGrid() then
	local f = CreateFrame("Frame")
	f:RegisterEvent("ADDON_LOADED")
	f:RegisterEvent("PLAYER_LOGIN")
	f:SetScript("OnEvent", function()
		if SkinGrid() then
			f:UnregisterAllEvents()
			f:SetScript("OnEvent", nil)
			SkinGrid = nil
		end
	end)
end

And here's a screen to explain what I mean. As you can see, only the first "UnitButton" is skinned.

[Phanx]

That's because only the first frame exists when the addons load (eg. one frame for yourself, since the game doesn't yet know you're in a group).

You need to figure out what Grid's equivalent of :RegisterFrame is, and adjust the code accordingly. This is the function that is called when a new frame is created, and adds all of the visible elements (such as the frame's backdrop) and indicators to the frame.

[lerb]

Alright! I'll hunt through grid's Lua-files when I get the time. If I do and find myself still clueless, could I ask you for more help? I like it when I have to do stuff myself so I learn, but still a newbie

[Phanx]

Isn't that the point of this forum? If you have a question, ask it. If you don't understand the answer, ask another question. If you need help with code, post the code and explain what you need help with.

[lerb]

Yeah, with the risk of feeling annoying You know, when someone explains something and you think you undertand, but it turns out you don't so you have to ask them to explain even more. It's a rare thing that's going on in this forum, where people are so helpful.

[lerb]

I found this line in "GridFrame.lua"

function GridFrame:RegisterFrame(frame)
	self:Debug("RegisterFrame", frame:GetName())

	self.registeredFrameCount = (self.registeredFrameCount or 0) + 1
	self.registeredFrames[frame:GetName()] = self:InitializeFrame(frame)
	self:UpdateFrameUnits()
end

Changing it to this did exactly what I wanted! Thanks for the hints Phanx

function GridFrame:RegisterFrame(frame)
	self:Debug("RegisterFrame", frame:GetName())
	
	CreateBorderLight(frame, LeUI.media.bordersize, LeUI.bordercolor, LeUI.bordercolor, LeUI.bordercolor, 2)

	self.registeredFrameCount = (self.registeredFrameCount or 0) + 1
	self.registeredFrames[frame:GetName()] = self:InitializeFrame(frame)
	self:UpdateFrameUnits()
end

[Phanx]

You should generally avoid modifying addon files directly. The only reason to do so would be if the function you need to modify is declared as a local, and cannot be accessed from outside the file.

hooksecurefunc(GridFrame, "RegisterFrame", function(frame)
	CreateBorderLight(frame, LeUI.media.bordersize, LeUI.bordercolor, LeUI.bordercolor, LeUI.bordercolor, 2)
end

[lerb]

Alright. But where should I be putting that hooksecurefunc? I've tried adding it to the SkinGrid addon but can't get it to work.

Here's how the code looks now, but it doesn't do anything since I don't have anything in the Grid_AddBorder funcion. If you could point out where to add it that would be awesome

local function SkinGrid()
    local GridFrame = Grid and Grid:GetModule("GridFrame")
    if GridFrame and GridFrame.registeredFrames then
        -- print("Adding borders to Grid frames")
 
        local function Grid_AddBorder(f)
            
        end
 
        local o = GridFrame.RegisterFrame
        GridFrame.RegisterFrame = function(self, f)
            o(self, f)
            Grid_AddBorder(f)
        end
 
        return true
    end
end
 
if not SkinGrid() then
    local f = CreateFrame("Frame")
    f:RegisterEvent("ADDON_LOADED")
    f:RegisterEvent("PLAYER_LOGIN")
    f:SetScript("OnEvent", function()
        if SkinGrid() then
            f:UnregisterAllEvents()
            f:SetScript("OnEvent", nil)
            SkinGrid = nil
        end
    end)
end

[unlimit]

Originally Posted by lerb Alright. But where should I be putting that hooksecurefunc? I've tried adding it to the SkinGrid addon but can't get it to work.

I'd assume that if you're adding the hooksecurefunc to your addon, you should confirm that Grid2 actually loads up before your addon does by putting it as a dependency in your TOC file, then adding the hooksecurefunc before your other two functions.

[lerb]

Thanks for the hints unlimit. I did what you said, but still there seems to be something I'm missing. To make it simple, I tried using this code;

local function SkinGrid()
    local GridFrame = Grid and Grid:GetModule("GridFrame")
    if GridFrame and GridFrame.registeredFrames then
        -- print("Adding borders to Grid frames")
        
        local function Grid_AddBorder(f)
            CreateBorderLight(frame, LeUI.media.bordersize, LeUI.bordercolor, LeUI.bordercolor, LeUI.bordercolor, 2)
        end
        
        hooksecurefunc(GridFrame, "RegisterFrame", function(frame)
            Grid_AddBorder(f)
        end)
 
        return true
    end
end
 
if not SkinGrid() then
    local f = CreateFrame("Frame")
    f:RegisterEvent("ADDON_LOADED")
    f:RegisterEvent("PLAYER_LOGIN")
    f:SetScript("OnEvent", function()
        if SkinGrid() then
            f:UnregisterAllEvents()
            f:SetScript("OnEvent", nil)
            SkinGrid = nil
        end
    end)
end

Sadly, that doesn't do anything. I'm not sure about how hooksecurefunc works, but I'm guessing out of the blue that this code is something like this; "When grid spawns a new frame, this hooksecurefunc 'senses' it and fires my Grid_AddBorder function, which is a command that tells !beautycase to add a border to the frame spawned by grid".

Please, correct me if I'm wrong. Which I probably am

[unlimit]

Signature: hooksecurefunc([table,] "function", hookfunc) Arguments: table - A table object that contains the function to be hooked (table) function - The name of the function to be hooked (string) hookfunc - The function to be called each time the original function is called (function)

So every time, in your case, GridFrame:RegisterFrame is called, the function you specify will also run along side of it. (I think.)

Also, I think half of the advice I gave you is also wrong. <.< Setting it as a dependency is right, but honestly I don't really know where to put the hooksecurefunc now that I look at the code? Possibly just:

local function SkinGrid()
    local GridFrame = Grid and Grid:GetModule("GridFrame")
    if GridFrame and GridFrame.registeredFrames then
        
        hooksecurefunc(GridFrame, "RegisterFrame", function(frame)
            CreateBorderLight(frame, LeUI.media.bordersize, LeUI.bordercolor, LeUI.bordercolor, LeUI.bordercolor, 2)
            print("Adding borders to Grid frame")
        end)
 
        return true
    end
end
 
if not SkinGrid() then
    local f = CreateFrame("Frame")
    f:RegisterEvent("ADDON_LOADED")
    f:RegisterEvent("PLAYER_LOGIN")
    f:SetScript("OnEvent", function()
        if SkinGrid() then
            f:UnregisterAllEvents()
            f:SetScript("OnEvent", nil)
            SkinGrid = nil
        end
    end)
end

>.> I'm still learning here too.

[lerb]

Using that code does not print the message, and gives this error;

Message: Interface\AddOns\!Beautycase\!Beautycase.lua:11: attempt to call method 'CreateTexture' (a nil value)
Time: 05/14/12 19:45:55
Count: 6
Stack: Interface\AddOns\!Beautycase\!Beautycase.lua:11: in function `CreateBorderLight'
Interface\AddOns\SkinGrid\SkinGrid.lua:6: in function <Interface\AddOns\SkinGrid\SkinGrid.lua:5>
[C]: in function `RegisterFrame'
Interface\AddOns\Grid\GridFrame.lua:33: in function `InitialConfigFunction'
Interface\AddOns\Grid\GridLayout.lua:122: in function `GridLayout_InitialConfigFunction'
Interface\AddOns\Grid\GridLayout.lua:144: in function `method'
Interface\FrameXML\RestrictedFrames.lua:721: in function <Interface\FrameXML\RestrictedFrames.lua:713>
[C]: ?
[C]: ?
Interface\FrameXML\RestrictedFrames.lua:740: in function `CallMethod'
[string "		RegisterUnitWatch(self)..."]:14: in function <[string "		RegisterUnitWatch(self)..."]:1>
(tail call): ?
...
(tail call): ?
[C]: ?
[string "safecall Dispatcher[1]"]:9: in function <[string "safecall Dispatcher[1]"]:5>
(tail call): ?
...AddOns\Bartender4\libs\AceAddon-3.0\AceAddon-3.0.lua:543: in function `EnableAddon'
...AddOns\Bartender4\libs\AceAddon-3.0\AceAddon-3.0.lua:636: in function <...AddOns\Bartender4\libs\AceAddon-3.0\AceAddon-3.0.lua:621>
[C]: in function `LoadAddOn'
Interface\FrameXML\UIParent.lua:274: in function `UIParentLoadAddOn'
Interface\FrameXML\UIParent.lua:297: in function `CombatLog_LoadUI'
Interface\FrameXML\UIParent.lua:616: in function <Interface\FrameXML\UIParent.lua:582>

Locals: self = <table> {
 SetDefaultModuleLibraries = <function> defined @Interface\AddOns\Bartender4\libs\AceAddon-3.0\AceAddon-3.0.lua:383
 UpdateFrameUnits = <function> defined @Interface\AddOns\Grid\GridFrame.lua:1320
 EnableModule = <function> defined @Interface\AddOns\Bartender4\libs\AceAddon-3.0\AceAddon-3.0.lua:348
 modules = <table> {
 }
 CancelTimer = <function> defined @Interface\AddOns\Grid\Libs\AceTimer-3.0\AceTimer-3.0.lua:311
 IterateEmbeds = <function> defined @Interface\AddOns\Bartender4\libs\AceAddon-3.0\AceAddon-3.0.lua:458
 ResetAllFrames = <function> defined @Interface\AddOns\Grid\GridFrame.lua:1291
 RegisterFrame = <function> defined =[C]:-1
 SendMessage_UpdateFrameUnits = <function> defined @Interface\AddOns\Grid\GridFrame.lua:1239
 debugging = false
 InvertBarColor = <function> defined @Interface\AddOns\Grid\GridFrame.lua:1312
 defaultModuleState = true
 IsEnabled = <function> defined @Interface\AddOns\Bartender4\libs\AceAddon-3.0\AceAddon-3.0.lua:467
 ScheduleTimer = <function> defined @Interface\AddOns\Grid\Libs\AceTimer-3.0\AceTimer-3.0.lua:276
 DisableModule = <function> defined @Interface\AddOns\Bartender4\libs\AceAddon-3.0\AceAddon-3.0.lua:366
 RegisterMessage = <function> defined @Interface\AddOns\Bartender4\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:118
 UnregisterMessage = <function> defined @Interface\AddOns\Bartender4\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:181
 RegisterBucketEvent = <function> defined @Interface\AddOns\Grid\Libs\AceBucket-3.0\AceBucket-3.0.lua:213
 Grid_StatusLost = <function> defined @Interface\AddOns\Grid\GridFrame.lua:1466
 db = <table> {
 }
 LibSharedMedia_Update = <function> defined @Interface\AddOns\Grid\GridFrame.lua:1243
 StatusForIndicator = <function> defined @Interface\AddOns\Grid\GridFrame.lua:1399
 SetEnabledState = <function> defined @Interface\AddOns\Bartender4\libs\AceAddon-3.0\AceAddon-3.0.lua:440
 enabledState = true
 RegisterEvent = <function> defined @Interface\AddOns\Bartender4\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:118
 IterateModules = <function> defined @Interface\AddOns\Bartender4\libs\AceAddon-3.0\AceAddon-3.0.lua:453
 UpdateOptionsMenu = <function> defined @Interface\AddOns\Grid\GridFrame.lua:1476
 registeredFrameCount = 6
 InitializeFrame = <function> defined @Interface\AddOns\Grid\GridFrame.lua:43
 InitialConfigFunction = <function> defined @Interface\AddOns\Grid\GridFrame.lua:32
 Enable = <function> defined @Interface\AddOns\Bartender4\libs\AceAddon-3.0\

I don't really know why. Here's the function in !beautycase that gave the error;

function CreateBorderLight(self, borderSize, R, G, B, ...)
    local uL1, uL2, uR1, uR2, bL1, bL2, bR1, bR2 = ...
    if (uL1) then
        if (not uL2 and not uR1 and not uR2 and not bL1 and not bL2 and not bR1 and not bR2) then
            uL2, uR1, uR2, bL1, bL2, bR1, bR2 = uL1, uL1, uL1, uL1, uL1, uL1, uL1
        end
    end
    if (not self.HasBorder) then
        self.Border = {}
        for i = 1, 8 do
            self.Border[i] = self:CreateTexture(nil, 'OVERLAY')
            self.Border[i]:SetParent(self)
            self.Border[i]:SetTexture(LeUI.media.borderlight)
            self.Border[i]:SetSize(borderSize,borderSize)
            self.Border[i]:SetVertexColor(R, G, B)
        end
        
        self.Border[1]:SetTexCoord(0, 1/3, 0, 1/3) 
        self.Border[1]:SetPoint('TOPLEFT', self, -(uL1 or 0), uL2 or 0)
 
        self.Border[2]:SetTexCoord(2/3, 1, 0, 1/3)
        self.Border[2]:SetPoint('TOPRIGHT', self, uR1 or 0, uR2 or 0)
 
        self.Border[3]:SetTexCoord(0, 1/3, 2/3, 1)
        self.Border[3]:SetPoint('BOTTOMLEFT', self, -(bL1 or 0), -(bL2 or 0))
 
        self.Border[4]:SetTexCoord(2/3, 1, 2/3, 1)
        self.Border[4]:SetPoint('BOTTOMRIGHT', self, bR1 or 0, -(bR2 or 0))
 
        self.Border[5]:SetTexCoord(1/3, 2/3, 0, 1/3)
        self.Border[5]:SetPoint('TOPLEFT', self.Border[1], 'TOPRIGHT')
        self.Border[5]:SetPoint('TOPRIGHT', self.Border[2], 'TOPLEFT')
 
        self.Border[6]:SetTexCoord(1/3, 2/3, 2/3, 1)
        self.Border[6]:SetPoint('BOTTOMLEFT', self.Border[3], 'BOTTOMRIGHT')
        self.Border[6]:SetPoint('BOTTOMRIGHT', self.Border[4], 'BOTTOMLEFT')
 
        self.Border[7]:SetTexCoord(0, 1/3, 1/3, 2/3)
        self.Border[7]:SetPoint('TOPLEFT', self.Border[1], 'BOTTOMLEFT')
        self.Border[7]:SetPoint('BOTTOMLEFT', self.Border[3], 'TOPLEFT')
 
        self.Border[8]:SetTexCoord(2/3, 1, 1/3, 2/3)
        self.Border[8]:SetPoint('TOPRIGHT', self.Border[2], 'BOTTOMRIGHT')
        self.Border[8]:SetPoint('BOTTOMRIGHT', self.Border[4], 'TOPRIGHT')
        
        local space
        if (borderSize >= 10) then
            space = 3.1
        else
            space = borderSize/3.5
        end
        
        self.Shadow = {}       
        for i = 1, 8 do
            self.Shadow[i] = self:CreateTexture(nil, 'BORDER')
            self.Shadow[i]:SetParent(self)
            self.Shadow[i]:SetTexture(LeUI.media.shadow)
            self.Shadow[i]:SetSize(borderSize, borderSize)  
            self.Shadow[i]:SetVertexColor(0, 0, 0, 1)
        end
        
        self.Shadow[1]:SetTexCoord(0, 1/3, 0, 1/3) 
        self.Shadow[1]:SetPoint('TOPLEFT', self, -(uL1 or 0)-space, (uL2 or 0)+space)
 
        self.Shadow[2]:SetTexCoord(2/3, 1, 0, 1/3)
        self.Shadow[2]:SetPoint('TOPRIGHT', self, (uR1 or 0)+space, (uR2 or 0)+space)
 
        self.Shadow[3]:SetTexCoord(0, 1/3, 2/3, 1)
        self.Shadow[3]:SetPoint('BOTTOMLEFT', self, -(bL1 or 0)-space, -(bL2 or 0)-space)
 
        self.Shadow[4]:SetTexCoord(2/3, 1, 2/3, 1)
        self.Shadow[4]:SetPoint('BOTTOMRIGHT', self, (bR1 or 0)+space, -(bR2 or 0)-space)
 
        self.Shadow[5]:SetTexCoord(1/3, 2/3, 0, 1/3)
        self.Shadow[5]:SetPoint('TOPLEFT', self.Shadow[1], 'TOPRIGHT')
        self.Shadow[5]:SetPoint('TOPRIGHT', self.Shadow[2], 'TOPLEFT')
 
        self.Shadow[6]:SetTexCoord(1/3, 2/3, 2/3, 1)
        self.Shadow[6]:SetPoint('BOTTOMLEFT', self.Shadow[3], 'BOTTOMRIGHT')
        self.Shadow[6]:SetPoint('BOTTOMRIGHT', self.Shadow[4], 'BOTTOMLEFT')
 
        self.Shadow[7]:SetTexCoord(0, 1/3, 1/3, 2/3)
        self.Shadow[7]:SetPoint('TOPLEFT', self.Shadow[1], 'BOTTOMLEFT')
        self.Shadow[7]:SetPoint('BOTTOMLEFT', self.Shadow[3], 'TOPLEFT')
 
        self.Shadow[8]:SetTexCoord(2/3, 1, 1/3, 2/3)
        self.Shadow[8]:SetPoint('TOPRIGHT', self.Shadow[2], 'BOTTOMRIGHT')
        self.Shadow[8]:SetPoint('BOTTOMRIGHT', self.Shadow[4], 'TOPRIGHT')
 
        self.HasBorder = true
    end
end

[unlimit]

Gee Wizz. D: Errors left and right. Yeah, like I said, I'm not really sure. Phanx or someone else who actually knows what they're doing is going to look at that code and think: "Nice job retard Unlimit!" the next time they get the chance! >.<

[lerb]

I feel exactly the same way every time I get help on this forum but still have to crawl back for more

[Phanx]

Well, here is one problem:

Originally Posted by lerb local function Grid_AddBorder(f) CreateBorderLight(frame, LeUI.media.bordersize, LeUI.bordercolor, LeUI.bordercolor, LeUI.bordercolor, 2)

Here is a more commented version of what should, theoretically, be working code:

local function SkinGrid()
    print("SkinGrid")
 
    -- IMPORTANT: If Grid2 gives each of its parts a cluttery global name,
    -- leave this line commented. Otherwise, uncomment it and adjust as
    -- needed:
 
    -- local GridFrame = Grid and Grid:GetModule("GridFrame")
 
    if GridFrame and GridFrame.registeredFrames then
        -- Grid is already loaded. Go ahead and skin the frames.
        print("Grid is loaded!")
 
        -- This function will be used to override the default border
        -- coloring method on a Grid frame.
        local function Grid_SetBackdropBorderColor(frame, r, g, b, a)
            if a and a == 0 then
                -- Instead of hiding the border when it's not showing a
                -- status, we want to reset it to the default color.
                frame:SetBorderColor()
            else
                -- Otherwise, we want to ignore the status's alpha value
                -- and always show the border at 100% opacity.
                frame:SetBorderColor(r, g, b)
            end
        end
 
        -- This function adds a border to a single frame.
        local function Grid_AddBorder(frame)
            -- Check if we already added a border to this frame:
            if not frame.SetBorderColor then
                -- Tell Grid the border is very small, so it will adjust
                -- the insets on the frame accordingly:
 
                -- IMPORTANT: You will need to find the equivalent
                -- function in Grid2 and change/uncomment this line:
 
                -- frame:SetBorderSize(0.1)
 
                -- Actually add the border:
                CreateBorderChat(frame,
                    LeUI.media.bordersize,
                    LeUI.bordercolor,
                    LeUI.bordercolor,
                    LeUI.bordercolor,
                    2)
 
                -- Hook the frame's border coloring method:
 
                -- IMPORTANT: Find out if this is necessary with Grid2
                -- and/or !Beautycase, and uncomment this line if so:
 
                -- frame.SetBackdropBorderColor = Grid_SetBackdropBorderColor
 
 
 
                -- Disable Grid's method for changing the border size:
 
                -- IMPORTANT: Find the equiavlent function in Grid2 and
                -- change/uncomment this line:
 
                -- frame.SetBorderSize = noop
            end
        end
 
        -- Loop through all of the frames that have already been created
        -- and add a border to each one:
        for frameName in pairs(GridFrame.registeredFrames) do
            Grid_AddBorder(_G[frameName])
            print("Added border to", frameName, "in SkinGrid.")
        end
 
        -- Hook Grid's RegisterFrame method to catch new frames when
        -- they are created in the future:
        hooksecurefunc(GridFrame, "RegisterFrame", function(frame)
            Grid_AddBorder(frame)
            print("Added border to", frame:GetName(), "in RegisterFrame hook.")
        end
 
        -- Return true, since the function successfully ran.
        return true
    end
end
 
if not SkinGrid() then
    print("Grid not loaded.")
    -- A nil return value indicates that Grid was not loaded yet,
    -- so we need to watch for that:
    local f = CreateFrame("Frame")
    f:RegisterEvent("ADDON_LOADED")
    f:RegisterEvent("PLAYER_LOGIN")
    f:SetScript("OnEvent", function()
        if SkinGrid() then
            -- Grid is now loaded, and has been skinned.
            -- Stop listening for addon loading events.
            print("Grid is now loaded. Stop listening.")
            f:UnregisterAllEvents()
            f:SetScript("OnEvent", nil)
            SkinGrid = nil
        else
            -- If SkinGrid() returned nil again, that means Grid is
            -- still not loaded, so we should keep listening.
            print("Grid is still not loaded. Keep listening.")
        end
    end)
end

Comments beginning with "IMPORTANT:" indicate sections that may need to be changed depending on how !Beautycase and/or Grid2 work internally. The code should work as-is, but if something doesn't work, those are the sections you should check first.

Not tested in-game, so may contain syntax errors.