flow-components icon indicating copy to clipboard operation
flow-components copied to clipboard

Discussion - More LitRenderer Support

Open oliveryasuna opened this issue 1 year ago • 2 comments
trafficstars

I thought it best to open this instead of a feature request, as this may require discussion first and multiple issues per feature...

One could argue that LitRenderer is preferred over ComponentRenderer. Better performance alone is a strong argument. Unless I am missing something, LitRenderer is not supported for Grid headers and footers, and TreeGrid hierarchy columns. Is there a particular reason why? I think it should be supported.

oliveryasuna avatar Feb 29 '24 02:02 oliveryasuna

That improved performance isn't super relevant for headers since you have a fixed number of those and they typically remain the same while the user interacts with the application. The main benefit is for cells in the body since there are many of them visible at the same time and they need to update rapidly as the user scrolls through the data.

Legioth avatar Feb 29 '24 10:02 Legioth

@Legioth Agreed, the main benefit would be for hierarchy columns. However, I feel it that it is bad practice to use ComponentRenderer, when avoidable. I have a rule for my team: If plaintext is not enough, and it can use LitRenderer over ComponentRenderer, it should use LitRenderer. More scalable for larger datasets or more columns.

oliveryasuna avatar Feb 29 '24 17:02 oliveryasuna