Fix Math.max returning -Infinity when columnIndices is empty array

[Nepomuk5665]

Summary

Fixes a bug in resizeRowsByTallestCell where passing an empty array [] for the columnIndices parameter causes all rows to be resized to -Infinity height.

Problem

When resizeRowsByTallestCell is called with columnIndices = []:

1. Line 161: columnIndicesArray becomes []
2. Line 162: tallestByColumns becomes [] (empty after map)
3. Line 163: Math.max(...[]) returns -Infinity

This causes all rows to have -Infinity height, which is incorrect behavior.

> Math.max(...[])
-Infinity

Solution

Add a length check before calling Math.max(). When columnIndicesArray is empty, the default value of tallest = 0 is preserved, which matches the expected behavior when no columns are specified.

Testing

This edge case was not covered by existing tests. The fix ensures consistent behavior:

• resizeRowsByTallestCell() → uses viewport columns (existing)
• resizeRowsByTallestCell([0, 1]) → uses specified columns (existing)
• resizeRowsByTallestCell([]) → now correctly returns 0 height (fixed)

[palantirtech]

Thanks for your interest in palantir/blueprint, @Nepomuk5665! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

[changelog-app]

Generate changelog in packages/table/changelog/@unreleased

Type (Select exactly one)

• Feature (Adding new functionality)
• Improvement (Improving existing functionality)
• Fix (Fixing an issue with existing functionality)
• Break (Creating a new major version by breaking public APIs)
• Deprecation (Removing functionality in a non-breaking way)
• Migration (Automatically moving data/functionality to a new system)

Description

Fix Math.max returning -Infinity when columnIndices is empty array

Check the box to generate changelog(s)

• Generate changelog entry