Add defensive checks for cases where maxHeight is empty string (#482)

* Fix assertion error when maxHeight is empty string

  When getComputedStyle().maxHeight returns an empty string (which can
  happen during DOM transitions, animations, or certain CSS edge cases),
  the existing check `maxHeightStyle !== 'none'` passes, but the
  subsequent call to estimateElementHeight() fails its assertion because
  empty string is falsy.

  Add a truthiness check to prevent calling estimateElementHeight() with
  an empty or falsy maxHeight value.

  Fixes assertion: "You called estimateElementHeight without a fallbackHeight"

* fix: add defensive check for error: Uncaught Error: Assertion Failed: You called estimateElement height without a fallbackHeight
    at assert$1 (assert.js:40:13)
    at estimateElementHeight (estimate-element-height.js:3:3)
    at StaticRadar._updateConstants (radar.js:340:36)
    at StaticRadar.update (radar.js:222:10)
    at radar.js:214:12
    at Array.execJob (index.js:30:7)
    at Scheduler.flush (index.js:92:11)
    at index.js:78:12
    at sentryWrapped (helpers.js:117:17)

Author: johanrd

vertical-collection/addon/-private/data-view/radar/radar.js

@@ -410,7 +410,7 @@ export default class Radar {
       const maxHeightStyle =
         window.getComputedStyle(_scrollContainer).maxHeight;
 
-      if (maxHeightStyle !== 'none') {
+      if (maxHeightStyle && maxHeightStyle !== 'none') {
         scrollContainerMaxHeight = estimateElementHeight(
           _scrollContainer.parentElement,
           maxHeightStyle,
@@ -419,7 +419,7 @@ export default class Radar {
     }
 
     const calculatedEstimateHeight =
-      typeof estimateHeight === 'string'
+      typeof estimateHeight === 'string' && estimateHeight
         ? estimateElementHeight(_itemContainer, estimateHeight)
         : estimateHeight;