I like what I'm seeing so far! (and it uses my lib, which is cool!)
Since you asked for feedback, after a cursory look I found two things:
1. Repetition in skins.lua:
Code: Select all
local framefont = love.graphics.newFont(10)
local buttonfont = love.graphics.newFont(10)
local progressbarfont = love.graphics.newFont(10)
...
local checkboxfont = love.graphics.newFont(10)
local columnlistheaderfont = love.graphics.newFont(10)
That's a lot of unnecessary allocations, since love.graphics.newFont isn't memoized (I think). This is equivalent, but faster and less memory-hungry:
Code: Select all
local font = love.graphics.newFont(10)
local framefont = font
local buttonfont = font
...
This also makes the code more DRY - if someone wants to change the font in all controls, they can do it in one single place.
Something similar happens with the colors: there is repetition.
Code: Select all
skin.controls.button_border_down_color = {143, 143, 143, 255}
skin.controls.button_border_nohover_color = {143, 143, 143, 255}
skin.controls.button_border_hover_color = {143, 143, 143, 255}
I'd rather have the repeated colors set up in one variable, like this:
Code: Select all
local secondaryColor = {143, 143, 143, 255}
...
skin.controls.button_border_down_color = secondaryColor
skin.controls.button_border_nohover_color = secondaryColor
skin.controls.button_border_hover_color = secondaryColor
2. I might have missed it, but I think there is no way to interact with this via keyboard/pad, right? You kinda need the mouse. If not, consider this a feature request.
That's all for now - mostly minor picks. Well done!