Split MainFrame into its own file

GnomishVendorShrinker.toc

@@ -22,10 +22,11 @@ LibItemSearch-1.0.lua
 
 frames\AltCurrency.lua
 frames\AltCurrencyItem.lua
+frames\MainFrame.lua
 frames\MerchantItem.lua
 frames\QtyPopout.lua
 frames\RowShader.lua
 frames\SearchField.lua
 
+Init.lua
 KnownScanner.lua
-GnomishVendorShrinker.lua

Init.lua

@@ -0,0 +1,54 @@
+
+local myname, ns = ...
+
+
+local ItemSearch = ns.LibItemSearch
+ns.LibItemSearch = nil
+
+
+local NUMROWS, SCROLLSTEP = 14, 5
+
+
+local function Hide(frame)
+	frame:Hide()
+	frame.Show = frame.Hide
+end
+
+
+function ns.OnLoad()
+	Hide(MerchantNextPageButton)
+	Hide(MerchantPrevPageButton)
+	Hide(MerchantPageText)
+
+
+	local GVS = ns.NewMainFrame()
+	GVS:SetWidth(315)
+	GVS:SetHeight(294)
+	GVS:SetPoint("TOPLEFT", 8, -67)
+	GVS:Hide()
+
+
+	-- Reanchor the buyback button, it acts weird when switching tabs otherwise...
+	MerchantBuyBackItem:ClearAllPoints()
+	MerchantBuyBackItem:SetPoint("BOTTOMRIGHT", -7, 33)
+
+
+	local function Show()
+		for i=1,12 do _G["MerchantItem"..i]:Hide() end
+		if GVS:IsShown() then GVS:GetScript("OnShow")(GVS, true) else GVS:Show() end
+	end
+	hooksecurefunc("MerchantFrame_UpdateMerchantInfo", Show)
+
+
+	hooksecurefunc("MerchantFrame_UpdateBuybackInfo", function()
+		GVS:Hide()
+		for i=1,12 do _G["MerchantItem"..i]:Show() end
+	end)
+
+
+	if MerchantFrame:IsVisible() and MerchantFrame.selectedTab == 1 then Show() end
+
+
+	-- Clean up our frame factories
+	for i,v in pairs(ns) do if i:match("^New") then ns[i] = nil end end
+end

frames/MainFrame.lua

@@ -6,30 +6,11 @@ local ItemSearch = ns.LibItemSearch
 ns.LibItemSearch = nil
 
 
-local NUMROWS, SCROLLSTEP = 14, 5
+local NUMROWS = 14
-
-
-local function Hide(frame)
-	frame:Hide()
-	frame.Show = frame.Hide
-end
-
-
-function ns.OnLoad()
-	Hide(MerchantNextPageButton)
-	Hide(MerchantPrevPageButton)
-	Hide(MerchantPageText)
 
 
+function ns.NewMainFrame()
 	local GVS = CreateFrame("frame", nil, MerchantFrame)
-	GVS:SetWidth(315)
-	GVS:SetHeight(294)
-	GVS:SetPoint("TOPLEFT", 8, -67)
-	GVS:SetScript("OnEvent", function(self, event, ...)
-		if self[event] then return self[event](self, event, ...) end
-	end)
-	GVS:Hide()
-
 
 	local rows = {}
 	for i=1,NUMROWS do
@@ -47,8 +28,7 @@ function ns.OnLoad()
 	end
 
 
-	local scrollbar = ns.NewScrollBar(GVS, 0, SCROLLSTEP)
+	local scrollbar = ns.NewScrollBar(GVS, 0, 5)
-	local offset = 0
 	function scrollbar:Refresh()
 		local offset = scrollbar:GetValue()
 		local n = GetMerchantNumItems()
@@ -64,19 +44,13 @@ function ns.OnLoad()
 			end
 		end
 		scrollbar:SetMinMaxValues(0, math.max(0, n_searchmatch - NUMROWS))
-		for i=row,NUMROWS do
+		for i=row,NUMROWS do rows[i]:Hide() end
-			rows[i]:Hide()
-		end
 	end
-	GVS.CURRENCY_DISPLAY_UPDATE = scrollbar.Refresh
-	GVS.BAG_UPDATE = scrollbar.Refresh
-	GVS.MERCHANT_UPDATE = scrollbar.Refresh
 
 
 	ns.MakeSearchField(GVS, scrollbar.Refresh)
 
 
-	local offset = 0
 	GVS:EnableMouseWheel(true)
 	GVS:SetScript("OnMouseWheel", function(self, value)
 		if value > 0 then
@@ -85,6 +59,7 @@ function ns.OnLoad()
 			scrollbar:Increment()
 		end
 	end)
+	GVS:SetScript("OnEvent", scrollbar.Refresh)
 	GVS:SetScript("OnShow", function(self, noreset)
 		local max = math.max(0, GetMerchantNumItems() - NUMROWS)
 		scrollbar:SetMinMaxValues(0, max)
@@ -96,34 +71,9 @@ function ns.OnLoad()
 		GVS:RegisterEvent("CURRENCY_DISPLAY_UPDATE")
 	end)
 	GVS:SetScript("OnHide", function()
-		GVS:UnregisterEvent("BAG_UPDATE")
+		GVS:UnregisterAllEvents()
-		GVS:UnregisterEvent("MERCHANT_UPDATE")
-		GVS:UnregisterEvent("CURRENCY_DISPLAY_UPDATE")
 		if StackSplitFrame:IsVisible() then StackSplitFrame:Hide() end
 	end)
 
-
+	return GVS
-	-- Reanchor the buyback button, it acts weird when switching tabs otherwise...
-	MerchantBuyBackItem:ClearAllPoints()
-	MerchantBuyBackItem:SetPoint("BOTTOMRIGHT", -7, 33)
-
-
-	local function Show()
-		for i=1,12 do _G["MerchantItem"..i]:Hide() end
-		if GVS:IsShown() then GVS:GetScript("OnShow")(GVS, true) else GVS:Show() end
-	end
-	hooksecurefunc("MerchantFrame_UpdateMerchantInfo", Show)
-
-
-	hooksecurefunc("MerchantFrame_UpdateBuybackInfo", function()
-		GVS:Hide()
-		for i=1,12 do _G["MerchantItem"..i]:Show() end
-	end)
-
-
-	if MerchantFrame:IsVisible() and MerchantFrame.selectedTab == 1 then Show() end
-
-
-	-- Clean up our frame factories
-	for i,v in pairs(ns) do if i:match("^New") then ns[i] = nil end end
 end