@bytebrand/fe-ui-core-autobahn 1.0.139 → 1.0.140

package/package.json

@@ -1,30 +1,30 @@
-{
-  "name": "@bytebrand/fe-ui-core-autobahn",
-  "version": "1.0.139",
-  "description": "auto.de md3 (Autobahn) design system — UI primitives, cards, chrome, VDP modules, theme CSS and locales. Ships raw TypeScript source (like fe-ui-core before it); each consumer's bundler compiles it.",
-  "main": "index.ts",
-  "license": "UNLICENSED",
-  "peerDependencies": {
-    "react": "^18.0.0",
-    "react-dom": "^18.0.0"
-  },
-  "dependencies": {
-    "@bytebrand/car-schema-selectors": "^2.0.7",
-    "@bytebrand/fe-financing": "^5.14.20",
-    "classnames": "2.2.6",
-    "clsx": "^2.1.1",
-    "d3-format": "1.4.5",
-    "js-cookie": "^2.2.0",
-    "lodash": "^4.17.21",
-    "lucide-react": "^0.288.0",
-    "mobx-react": "^7.6.0",
-    "moment": "^2.29.1",
-    "qs": "^6.6.0",
-    "react-device-detect": "^2.1.2",
-    "react-grid-system": "^5.0.2",
-    "react-lazyload": "^3.2.0",
-    "react-scroll": "^1.9.3",
-    "react-slick": "^0.29.0",
-    "slick-carousel": "1.8.1"
-  }
-}
+{
+  "name": "@bytebrand/fe-ui-core-autobahn",
+  "version": "1.0.140",
+  "description": "auto.de md3 (Autobahn) design system — UI primitives, cards, chrome, VDP modules, theme CSS and locales. Ships raw TypeScript source (like fe-ui-core before it); each consumer's bundler compiles it.",
+  "main": "index.ts",
+  "license": "UNLICENSED",
+  "peerDependencies": {
+    "react": "^18.0.0",
+    "react-dom": "^18.0.0"
+  },
+  "dependencies": {
+    "@bytebrand/car-schema-selectors": "^2.0.7",
+    "@bytebrand/fe-financing": "^5.14.20",
+    "classnames": "2.2.6",
+    "clsx": "^2.1.1",
+    "d3-format": "1.4.5",
+    "js-cookie": "^2.2.0",
+    "lodash": "^4.17.21",
+    "lucide-react": "^0.288.0",
+    "mobx-react": "^7.6.0",
+    "moment": "^2.29.1",
+    "qs": "^6.6.0",
+    "react-device-detect": "^2.1.2",
+    "react-grid-system": "^5.0.2",
+    "react-lazyload": "^3.2.0",
+    "react-scroll": "^1.9.3",
+    "react-slick": "^0.29.0",
+    "slick-carousel": "1.8.1"
+  }
+}

package/vdp/VdpModules.tsx

@@ -2240,28 +2240,29 @@ export const FinanceDetailsModal: React.FunctionComponent<FinanceDetailsModalPro
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [open, initialShowBreakdown]);
   // in-sheet "Finanzierungs-/Leasingdetails" toggle (mobile): expanding the breakdown alone left
-  // it below the fold — scroll the sheet body all the way down (same end state as opening via
-  // the page's details link) in one quick ~300ms ease-out. The click also skips
-  // .fin-modal--expanded's 0.35s min-height transition (see the toggle's onClick): a scroll
-  // running while the sheet is still growing gets clamped/retargeted every frame and stutters.
-  // Hand-rolled rAF instead of scrollTo({behavior:'smooth'}) so the duration is ours, not the
-  // browser's. Collapsing doesn't scroll.
+  // it below the fold — scroll the sheet body so the "Repräsentatives Beispiel" headline lands at
+  // the top of the body (not all the way down: that hid the headline and overshot). Runs
+  // alongside .fin-modal--expanded's own 0.35s min-height transition instead of waiting it out
+  // or disabling it: the target is re-read every frame and clamped to the current max scroll,
+  // so the growing sheet and the scroll finish together in one motion. Hand-rolled rAF instead
+  // of scrollTo({behavior:'smooth'}) so the duration is ours. Collapsing doesn't scroll.
   const revealBreakdownRef = React.useRef(false);
   React.useEffect(() => {
     if (!showBreakdown || !revealBreakdownRef.current) return;
     revealBreakdownRef.current = false;
-    const body = mobileBreakdownRef.current?.closest('.fin-modal-body') as HTMLElement | null | undefined;
-    if (!body) return;
+    const el = mobileBreakdownRef.current;
+    const body = el?.closest('.fin-modal-body') as HTMLElement | null | undefined;
+    if (!el || !body) return;
     const from = body.scrollTop;
-    const dist = body.scrollHeight - body.clientHeight - from;
-    if (dist <= 0) return;
-    const duration = 300;
+    const duration = 400;
     let start: number | null = null;
     let raf = 0;
     const step = (now: number): void => {
       if (start === null) start = now;
       const p = Math.min(1, (now - start) / duration);
-      body.scrollTop = from + dist * (1 - Math.pow(1 - p, 3));
+      const headlineTop = el.getBoundingClientRect().top - body.getBoundingClientRect().top + body.scrollTop;
+      const to = Math.max(from, Math.min(headlineTop - 8, body.scrollHeight - body.clientHeight));
+      body.scrollTop = from + (to - from) * (1 - Math.pow(1 - p, 3));
       if (p < 1) raf = window.requestAnimationFrame(step);
     };
     raf = window.requestAnimationFrame(step);
@@ -3443,9 +3444,6 @@ export const FinanceDetailsModal: React.FunctionComponent<FinanceDetailsModalPro
                   type="button"
                   onClick={() => {
                     revealBreakdownRef.current = !showBreakdown;
-                    // expanding: grow the sheet to full height at once (no min-height
-                    // transition) so the scroll-down above runs on a settled layout
-                    if (!showBreakdown && sheetRef.current) sheetRef.current.style.transition = 'none';
                     setShowBreakdown((v) => !v);
                   }}
                   style={{ ...expanderBtnStyle, justifyContent: 'center', padding: '2px 0 20px' }}>