From c00fbd275ded6d42363a31c370a970b6eb23e90b Mon Sep 17 00:00:00 2001
From: Adrian L Lange <adrianlund@gmail.com>
Date: Sun, 28 Apr 2013 01:42:29 +0200
Subject: [PATCH] Properly update the positions

---
 HabeebIt.lua |   44 ++++++++++++++++++++++++++------------------
 1 file changed, 26 insertions(+), 18 deletions(-)

diff --git a/HabeebIt.lua b/HabeebIt.lua
index effc036..cbb5132 100644
--- a/HabeebIt.lua
+++ b/HabeebIt.lua
@@ -5,8 +5,6 @@ local specializations = {}
 local collapsed = true
 local currentEncounterID
 
-local populateDown
-
 local Frame = CreateFrame('Frame', 'HabeebItFrame', BonusRollFrame)
 local Handle = CreateFrame('Button', 'HabeebItHandle', BonusRollFrame)
 
@@ -95,6 +93,26 @@ local function GetItemLine(index)
 	return Item
 end
 
+local function UpdatePosition()
+	local populateDown = Frame:GetBottom() > (UIParent:GetHeight() / 1.5)
+
+	Frame:ClearAllPoints()
+	if(populateDown) then
+		Frame:SetPoint('TOPLEFT', BonusRollFrame, 'TOPRIGHT')
+	else
+		Frame:SetPoint('BOTTOMLEFT', BonusRollFrame, 'BOTTOMRIGHT')
+	end
+
+	for index, button in pairs(items) do
+		button:ClearAllPoints()
+		if(populateDown) then
+			button:SetPoint('TOP', 0, (6 + ((index - 1) * 48)) * -1)
+		else
+			button:SetPoint('BOTTOM', 0, 6 + ((index - 1) * 48))
+		end
+	end
+end
+
 local function PopulateList()
 	local numItems = 0
 	for index = 1, EJ_GetNumLoot() do
@@ -108,13 +126,6 @@ local function PopulateList()
 			Item.Slot:SetText(slot)
 			Item.Class:SetText(itemClass)
 
-			Item:ClearAllPoints()
-			if(populateDown) then
-				Item:SetPoint('TOP', 0, (6 + ((numItems - 1) * 48)) * -1)
-			else
-				Item:SetPoint('BOTTOM', 0, 6 + ((numItems - 1) * 48))
-			end
-
 			Item.itemID = itemID
 			Item.itemLink = itemLink
 
@@ -129,6 +140,10 @@ local function PopulateList()
 	else
 		Frame.Empty:Show()
 	end
+
+	if(Frame:IsShown()) then
+		UpdatePosition()
+	end
 end
 
 local function UpdateSpecializations(currentIndex)
@@ -233,15 +248,6 @@ Frame:SetScript('OnEvent', function(self, event, ...)
 					CreateSpecializationTabs(self)
 				end
 
-				populateDown = self:GetBottom() > (UIParent:GetHeight() / 1.5)
-
-				self:ClearAllPoints()
-				if(populateDown) then
-					self:SetPoint('TOPLEFT', BonusRollFrame, 'TOPRIGHT')
-				else
-					self:SetPoint('BOTTOMLEFT', BonusRollFrame, 'BOTTOMRIGHT')
-				end
-
 				InitializeList(GetSpecialization())
 			else
 				print('|cffff8080HabeebIt:|r Found an unknown spell [' .. spellID .. ']. Please report this!')
@@ -256,6 +262,8 @@ Frame:SetScript('OnEvent', function(self, event, ...)
 		self:RegisterEvent('SPELL_CONFIRMATION_TIMEOUT')
 		self:RegisterEvent('EJ_LOOT_DATA_RECIEVED')
 
+		self:SetScript('OnShow', UpdatePosition)
+		self:SetPoint('BOTTOMLEFT', BonusRollFrame, 'BOTTOMRIGHT')
 		self:SetSize(338, 76)
 		self:SetFrameLevel(10)
 		self:Hide()
-- 
1.7.9.5