@ticketboothapp/booking 1.2.39 → 1.2.41

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ticketboothapp/booking",
-  "version": "1.2.39",
+  "version": "1.2.41",
   "private": false,
   "sideEffects": [
     "**/*.css",

package/src/components/booking/BookingFlow.tsx

@@ -1423,26 +1423,11 @@ export function BookingFlow({
       return null;
     };
 
-    const detectedScrollParent = findScrollParent(el);
-    const explicitScrollContainer = contentRef?.current ?? null;
-    const isElementScrollable = (node: HTMLElement | null): node is HTMLElement => {
-      if (!node) return false;
-      const style = getComputedStyle(node);
-      const allowsScroll = style.overflowY === 'auto' || style.overflowY === 'scroll' || style.overflowY === 'overlay';
-      return allowsScroll && node.scrollHeight > node.clientHeight + 1;
-    };
-
-    // Deterministic host-aware choice:
-    // 1) explicit contentRef when it is truly scrollable
-    // 2) nearest detected scroll parent when it is truly scrollable
-    // 3) window scroll
-    const scrollParent =
-      !useWindowScroll && isElementScrollable(explicitScrollContainer)
-        ? explicitScrollContainer
-        : !useWindowScroll && isElementScrollable(detectedScrollParent)
-        ? detectedScrollParent
-        : null;
-    const scrollTarget = scrollParent ?? (typeof window !== 'undefined' ? window : null);
+    const scrollParent = findScrollParent(el);
+    const scrollTarget =
+      useWindowScroll || !scrollParent
+        ? (typeof window !== 'undefined' ? window : null)
+        : scrollParent;
 
     let ticking = false;
     const COOLDOWN_MS = 600; // After a state change, ignore reverse changes for this long (covers 0.25s collapse animation + layout settle)

package/src/components/booking/ItineraryBox.tsx

@@ -166,36 +166,8 @@ export function ItineraryBox({
   const handlePickupLocationClick = () => {
     const pickupSection = document.getElementById('pickup-location-section');
     if (!pickupSection) return;
-
-    const findScrollParent = (node: HTMLElement): HTMLElement | null => {
-      let parent = node.parentElement;
-      while (parent) {
-        const { overflowY } = getComputedStyle(parent);
-        if (overflowY === 'auto' || overflowY === 'scroll' || overflowY === 'overlay') return parent;
-        parent = parent.parentElement;
-      }
-      return null;
-    };
-
-    const scrollParent = useWindowScroll ? null : findScrollParent(pickupSection);
-    const itineraryBox = document.querySelector('[class*="sticky top-"]');
-    const itineraryHeight = itineraryBox?.getBoundingClientRect().height ?? 0;
-    // Extra offset so sticky header doesn't block the pickup section (mobile needs more clearance)
-    const STICKY_HEADER_CLEARANCE = isMobile ? 280 : 270;
-    const targetOffsetFromTop = itineraryHeight + 16 + STICKY_HEADER_CLEARANCE;
-
-    if (scrollParent) {
-      const elementPosition = pickupSection.getBoundingClientRect().top;
-      const scrollDelta = elementPosition - targetOffsetFromTop;
-      scrollParent.scrollTo({
-        top: Math.max(0, scrollParent.scrollTop + scrollDelta),
-        behavior: 'smooth',
-      });
-    } else if (typeof window !== 'undefined') {
-      const elementPosition = pickupSection.getBoundingClientRect().top + window.scrollY;
-      const scrollTop = Math.max(0, elementPosition - targetOffsetFromTop);
-      window.scrollTo({ top: scrollTop, behavior: 'smooth' });
-    }
+    // Let browser handle the active scroll container (window or nested dialog/content area).
+    pickupSection.scrollIntoView({ behavior: 'smooth', block: 'center', inline: 'nearest' });
   };
 
   const getDisplayLabel = (label: string): string => {