liferay / clay

A web implementation of the Lexicon Experience Language
http://clayui.com
Other
208 stars 467 forks source link

fix(@clayui/core): Fixes bug when rendering the last columns visibility cell for dynamic rendering #5822

Closed matuzalemsteles closed 1 month ago

matuzalemsteles commented 1 month ago

Follow-up #5817

I had only done the test for the case of static rendering but had not done the coverage for the case of dynamic rendering, an important point is that we do this as a speculation of the size of the items within a row but this value can change if necessary in the cell, I believe that the best thing is for the developer to avoid cases like this so as not to break other table behaviors along with collection, so it always passes the items that will be rendered. This is also great for recommending that they are pure functions and have no side effects.