Fix to consider the top and bottom edge insets when updating the skeleton layer height

[xpereta]

The .top property of skeletonPaddingInsets is being used to offset the position of the skeleton layer, but the .bottom property is left unused. As a result the skeleton layer has a larger height than expected.

To reproduce de issue using the example application we can change the header label to use a bigger font:

And then add this line to configure the skeletonPaddingInsets:

    lazy var titleLabel: UILabel = {
        let label = UILabel()
        
        label.text = "      "
        label.font = UIFont.systemFont(ofSize: 40)
        label.isSkeletonable = true
        label.skeletonPaddingInsets = UIEdgeInsets(top: 30, left: 0, bottom: 30, right: 0)
        label.linesCornerRadius = 5
        
        return label
    }()

The skeleton layer is larger than expected:

With the change in this PR, the .bottom property is used and the skeleton layer has the expected height:

Please tell me if I'd rather create an issue to discuss it and then add a line in the changelog if you eventually approve it.

Thanks!

[Juanpe]

Hi @xpereta, thanks for contributing :)

Simple but useful PR, go ahead with the CHANGELOG 👍

welcome aboard!

[xpereta]

I updated the PR to also consider the top edge inset to calculate the skeleton layer height.

[Juanpe]

Thanks for your first contribution to SkeletonView 👏