fix clamp sizing for long stacks

[walterbender]

We have had a long-standing issue with the calculation of the clamp size. When you have a long stack inside of a clamp, you will notice that the clamp is not quite large enough. It is a 0.25 pixel adjustment (at standard scale) per clamp slot to fix the issue.

[github-actions]

✅ All Jest tests passed! This PR is ready to merge.

[walterbender]

Before:

After:

[walterbender]

@pikurasa could you please test?

[pikurasa]

It works, and some things look better.

I did notice that at the end of a string of notes, it looks like this in your branch:

For reference, this is master:

So, in this case, it seems to overcompensate.

[walterbender]

Thx. How many notes were in the clamp?

[pikurasa]

Thx. How many notes were in the clamp?

There were eight.

It seems like it does up to three notes the best. After that, it begins to drift.

[walterbender]

hmm. digging deeper, I think the issue is a bit different than I thought. It seems that the mis-sizing of the clamp is only an issue when the block scale is 1.5 (the default size). All the other sizes are integers. So what happens is that the block positions are calculated to .5 pixel precision but are being rendered at full pixel positions. I need to think about a solution.

[walterbender]

@pikurasa It is not perfect but I think it is better than before.

[github-actions]

✅ All Jest tests passed! This PR is ready to merge.

[walterbender]

I think the only way to fix this properly is to ensure that all block sizes, when multiplied by 1.5, are still integers. So all block sizes must be even numbers.

[walterbender]

I am going to take a different approach to this.