From b5346e86080874c78095c871c9afc28fe85af070 Mon Sep 17 00:00:00 2001
From: Kevin Lyles <kevinlyles@gmail.com>
Date: Thu, 22 Oct 2009 23:24:28 -0500
Subject: [PATCH] weight list buttons created upon selecting class

---
 WeightsWatcher.xml |   17 ++++++-----------
 config.lua         |   38 +++++++++++++++++++++++++++++++-------
 2 files changed, 37 insertions(+), 18 deletions(-)

diff --git a/WeightsWatcher.xml b/WeightsWatcher.xml
index 6763bfc..c443f72 100644
--- a/WeightsWatcher.xml
+++ b/WeightsWatcher.xml
@@ -7,19 +7,14 @@
 			<Anchor point="LEFT"/>
 		</Anchors>
 	</FontString>
-	<!--virtual button used to allow for script generated buttons with class names-->
-	<Button name="classButton" virtual="true">
+	<!--virtual button used to allow for script generated buttons-->
+	<Button name="genericButton" virtual="true">
 		<Size>
 			<AbsDimension x="75" y="20"/>
 		</Size>
 		<Scripts>
-			<OnClick>
-				configClassSelect(self:GetName())
-			</OnClick>
 			<OnLoad>
-				local fontString = self:CreateFontString(nil, "OVERLAY", "defaultString")
-				fontString:SetText(classNames[self:GetName()])
-				self:SetFontString(fontString)
+				self:SetFontString(self:CreateFontString(nil, "OVERLAY", "defaultString"))
 			</OnLoad>
 		</Scripts>
 	</Button>
@@ -44,7 +39,7 @@
 			<!--The Class Selection Frame-->
 			<Frame name="$parentLeftPanel" parentKey="leftPanel">
 				<Size>
-					<AbsDimension x="100" y="210"/>
+					<AbsDimension x="100" y="350"/>
 				</Size>
 				<Anchors>
 					<Anchor point="LEFT"/>
@@ -62,9 +57,9 @@
 				</Backdrop>
 			</Frame>
 			<!--Right Frame for viewing the weights for each class-->
-			<Frame>
+			<Frame name="$parentRightPanel" parentKey="rightPanel" hidden="true">
 				<Size>
-					<AbsDimension x="300" y="400"/>
+					<AbsDimension x="300" y="350"/>
 				</Size>
 				<Anchors>
 					<Anchor point="RIGHT"/>
diff --git a/config.lua b/config.lua
index e30a99c..338a775 100644
--- a/config.lua
+++ b/config.lua
@@ -2,6 +2,8 @@ if not WeightsWatcher then
 	WeightsWatcher = AceLibrary("AceAddon-2.0"):new("AceEvent-2.0", "AceHook-2.1")
 end
 
+weightButtonTable = {}
+
 function commandHandler(msg)
 	open_config()
 end
@@ -20,14 +22,31 @@ function open_config()
 	end
 end
 
+--opens the right panel and loads the appropriate buttons
 function configClassSelect(classType)
-	for name, _ in pairs(ww_vars.weightsList[classType]) do
-		addWeightScale(name)
+	local counter = 1
+	--retrieve the list of weights and update any visible buttons to reflect the changes
+	--NOTE: this approach is used to minimize memory leakage
+	for weightName, _ in pairs(ww_vars.weightsList[classType]) do
+		--if our previously created button table isn't big enough, add new buttons
+		if #(weightButtonTable) < counter then
+			table.insert(weightButtonTable, CreateFrame("Button", nil, wwConfig.rightPanel, "genericButton"))
+			weightButtonTable[counter]:SetPoint("TOPLEFT", 5, 15 - 20 * counter)
+		end
+		weightButtonTable[counter]:SetText(weightName)
+		weightButtonTable[counter]:SetScript("OnClick",
+			function()
+				print("Selecting weights not yet implemented")
+			end)
+		weightButtonTable[counter]:Show()
+		counter = counter + 1
 	end
-end
-
-function addWeightScale(name)
-	print(name)
+	--if we have any remaining buttons, hide them
+	while counter <= #(weightButtonTable) do
+		weightButtonTable[counter]:Hide()
+		counter = counter + 1
+	end
+	wwConfig.rightPanel:Show()
 end
 
 --loads the various class buttons onto the config frame
@@ -36,8 +55,13 @@ function loadClassButtons()
 	local i = 1
 	--creates a button for each class available in weightsList
 	for class, _ in pairs(ww_vars.weightsList) do
-		local newButton = CreateFrame("Button", class, wwConfig.leftPanel, "classButton")
+		local newButton = CreateFrame("Button", class, wwConfig.leftPanel, "genericButton")
 		newButton:SetPoint("TOPLEFT", 5, 15 - i * 20)
+		newButton:SetText(classNames[class])
+		newButton:SetScript("OnClick",
+			function()
+				configClassSelect(class)
+			end)
 		i = i + 1
 	end
 end
-- 
1.7.9.5