glide-data-grid icon indicating copy to clipboard operation
glide-data-grid copied to clipboard

Fix incorrect calculation of DataEditor height.

Open infalmo opened this issue 8 months ago • 1 comments

infalmo avatar May 11 '25 01:05 infalmo

I believe sampling only 10 rows as opposed to all rows was an intentional performance optimization, as summing heights for thousands or millions of rows is an issue.

BrianHung avatar Jun 18 '25 07:06 BrianHung

Yep, I think we can close this since its an intended optimisation. But maybe we could use a slightly higher value, e.g. 50 or 100 rows, to get a better approximation without adding to much computation overhead.

lukasmasuch avatar Jun 25 '25 23:06 lukasmasuch