collapse: remove overflow hidden even if height does not match perfect

[gehrisandro]

Sometimes it happens, that the overflow: hidden style remains on the element after it has expanded.

The reason is, that there is a condition checking that the element height as reached the full height, but sometimes the values do not match perfectly. For example, full is 100.5 but el.getBoundingClientRect().height returns 100.4765625.

This PR changes the condition to allow a difference up to 1px.

To be honest, I am not sure, if the condition is necessary at all.

[ekwoka]

I believe it has to do with the fact that function is called at multiple steps, but not totally sure. I think there are more modern features that can make this actually much easier and simpler, but won't satisfy older targets (Animation API, Animating Grid Rows).

[calebporzio]

Thanks!

[gehrisandro]

Thanks!

Thank you for merging 🚀

[bb]

I encountered the same issue today and can confirm that this PR fixes it. Thank you @gehrisandro!
Looking forward to the release. 😊