[goldbishop]

Originally Posted by Digital_Utopia

Most of it looks alright, but the following has got me scratching my head (in yellow)

function FrameworkReportWindow:Refresh()
self:PerformLayout();
end

function FrameworkReportWindow:PerformLayout()
self:Layout( { } );
end

function FrameworkReportWindow:Layout( args )
local width, height = self:GetSize();
end

When you make functions inside a class, they're generally made like so:

function self:Refresh()
     self:PerformLayout();
end

Think of those as stationary templates....till i get to the point of actually implementation.