ni / nimble

The NI Nimble Design System
https://nimble.ni.dev
MIT License
29 stars 9 forks source link

Consider moving row height calculation from virtualizer to table (and monitoring for token changes) #2212

Open msmithNI opened 1 week ago

msmithNI commented 1 week ago

🧹 Tech Debt

Currently the table virtualizer.ts computes rowHeight repeatedly when needed, based on the controlHeight and borderWidth token values. KeyboardNavigationManager also relies on row height indirectly (it uses pageSize which is a function of row height).