Autoheight widgets miscalculating height

[ranbena]

Issue Summary

Comparing version 7 with current master, it's apparent that autoheight table visualization widgets are higher now by 1 row.

Version 7	Master branch

How come?

#3830 added a 10px bottom padding to tables but with -10px margin to compensate*.
Thing is, autoheight is calculated with getBoundingClientRect() which understandably includes padding but not margin, thus adding 10px to the visualization height - resulting in an additional grid row.

* Mac only - therefore not detected by e2e tests 😓

[ranbena]

I'm thinking we should revert #3830 - it's a workaround for a browser bug (btw, happens to me on GH code blocks as well..) and I feel no obligation to keep it.

@arikfr @kravets-levko @gabrieldutra any better ideas?

[arikfr]

@ranbena while I agree with your sentiment towards #3830, the problem is that sometimes it really blocks the content and makes it impossible to read. I couldn't reliably reproduce it and in most cases it was fine, but I can see how frustrating it can be when it does happen.