don't hardcode row height, or enforce it

[severo]

I'm not sure if it's possible or easy to change, but we currently hardcode the row height:

hightable/src/components/HighTable/HighTable.tsx

Line 24 in c51b500

const rowHeight = 33 // row height px

But we don't enforce it (try changing the font size)

Also, ideally, we should adapt to the row height defined in the CSS, if any. So: measure it instead of hardcode it.

[platypii]

Yea this is one of the bigger limitations currently. It's a hard problem though.

I would love if we could support measuring the row height. Or even more ambitiously, to support variable row height per row. But the virtual scrolling math currently requires fixed height row. We will need to use more tricks if we want to preserve the magic and smoothness of the virtual scrolling with dynamic row heights. Not impossible but tricky to get right.

You're probably right that as long as we require a fixed height (which we currently do, or else you'll get weird scroll effects), we should probably enforce the height. It would be nice if it was configurable too.

[severo]

somewhat related to #241.