Issue with autogeneration script?

This script generates this:

As you can see, they are overlapping. How why is this?

It prints this, btw: 0 line 0 positionOnLine 0 line 1 positionOnLine 0 line 2 positionOnLine 1 line 3 positionOnLine 1 line 0 positionOnLine 1 line 1 positionOnLine 1 line 2 positionOnLine

--[[Scripted by WolfgangVonPrinz]]--
local frame = script.Parent
local genericFrame = script.templateFrame
local data = workspace.Data.MeshData

function findPosition(i)
    local positionOnLine = i % 4== 0 and 3 or (i % 4) - 1   
    local line = math.floor(i / 4)
    print(line .. " line")
    print(positionOnLine .. " positionOnLine")
    return UDim2.new(0, 5 + (positionOnLine * 115), 0, 5 + (line * 120) and positionOnLine ~= 3 or ((line - 1) * 120) + 5)
end

for i,v in pairs(data:GetChildren())do
    local position = findPosition(i)
    local Image = genericFrame:Clone()
    Image.Parent = frame
    Image.Position = position
    Image.PointPrice.Value = v.Price.Value
    Image.Image = "rbxassetid://" .. tostring(v.DecalId)
end