fix(layout): correct isLayoutCalculated always returning true (#2642)(by @spsaucier)

* fix(layout): correct isLayoutCalculated always returning true

The `isLayoutCalculated` derived value uses an OR condition to check
whether `containerHeight` has been provided:

    if (containerHeight !== null || containerHeight !== undefined)

This expression is always true for any value — if containerHeight is
null, the `!== undefined` branch is true, and vice versa. The intended
check should use AND:

    if (containerHeight !== null && containerHeight !== undefined)

Without this fix, `isLayoutCalculated` returns true before layout
measurements complete, allowing `evaluatePosition` to run with
`containerHeight = INITIAL_LAYOUT_VALUE (-999)`. This produces
incorrect detent positions and can cause the sheet's content panel to
animate to an off-screen position while the backdrop renders correctly
(since both are derived from the same animatedPosition, but the
incorrect initial animation target can leave the body off-screen after
the "completed" callback fires and the snap-point-change correction
races with the mount animation).

The second condition (`!== INITIAL_LAYOUT_VALUE`) still serves as the
primary guard, but the first condition should be logically correct so
it doesn't silently pass for null/undefined containerHeight values.

* fix: merged container height logic to avoid early setting to true

---------

Co-authored-by: Stephen Saucier <hulaman345@gmail.com>
Co-authored-by: gorhom <gorhom.dev@gmail.com>

Author: 3 people

src/components/bottomSheet/BottomSheet.tsx

@@ -225,12 +225,12 @@ const BottomSheetComponent = forwardRef<BottomSheet, BottomSheetProps>(
     const isLayoutCalculated = useDerivedValue(() => {
       let isContainerHeightCalculated = false;
       const { containerHeight, handleHeight } = animatedLayoutState.get();
-      //container height was provided.
-      if (containerHeight !== null || containerHeight !== undefined) {
-        isContainerHeightCalculated = true;
-      }
-      // container height did set.
-      if (containerHeight !== INITIAL_LAYOUT_VALUE) {
+      //container height was provided and set not to initial value
+      if (
+        containerHeight !== null &&
+        containerHeight !== undefined &&
+        containerHeight !== INITIAL_LAYOUT_VALUE
+      ) {
         isContainerHeightCalculated = true;
       }