@depict-ai/react-ui 1.3.1 → 1.3.3

package/dist/index.d.ts

@@ -4,7 +4,7 @@ import React, { ReactNode, PropsWithChildren, ReactElement } from "react";
 import { DPC } from "@depict-ai/dpc";
 import { Locale } from "@depict-ai/ui/locales";
 import { ModernDisplayWithPageUrl } from "@depict-ai/ui/src";
-export const version = "1.3.1";
+export const version = "1.3.3";
 interface PLPConfig<T extends Display> extends BaseProductListingConfig {
     /**
      * The product card to use for rendering products.
@@ -511,4 +511,5 @@ export const Navigation: {
     nextRouterNavigation: (router: import("next/router").NextRouter) => import("ui/components/DepictProvider").NavigateFunction;
 };
 export { CategoryTitle, embedded_num_products as EmbeddedNumProducts } from "@depict-ai/ui";
+export type { PageURLCreator } from "@depict-ai/ui";
 export type { Locale } from "@depict-ai/ui/locales";

package/dist/main.js

@@ -31,7 +31,7 @@ $parcel$export(module.exports, "version", () => $db7b67857900028b$export$83d89fb
 $parcel$export(module.exports, "Navigation", () => $b742053c966024b4$export$b8a61e5c71402559);
 /*! Depict.ai search and product listing react SDK wrapper */ // This is where the actual source of your React stuff should live
 // Note, if you add an export here, also add one in src-server/main.ts
-const $db7b67857900028b$export$83d89fbfd8236492 = "1.3.1";
+const $db7b67857900028b$export$83d89fbfd8236492 = "1.3.3";
 
 
 
@@ -193,11 +193,14 @@ function $6319748b4f0426f0$export$6453cbc34515a94d() {
     const are_on_search_or_category_page = ()=>(0, $b4d0d98857ed9934$export$775e6b3892b5ad52)() || (0, $b4d0d98857ed9934$export$30c8a12df09be0e4)();
     if ($6319748b4f0426f0$var$set) return; // Just being safe
     $6319748b4f0426f0$var$set = true;
-    let should_use_old_scroll = false;
+    // We used to initialise this variable with false which in some cases (scroll to top button in FloatingButtons) would cause the scroll to be blocked when it shouldn't (see https://depictaiworkspace.slack.com/archives/C058KQ3MU8Z/p1686141321127089?thread_ts=1686134246.109689&cid=C058KQ3MU8Z)
+    // I don't know why we'd want to initialise it with false? The only reason I can see is in the hopes of scroll restoration working on initial page load, but it basically never does due to react rendering and executing our component too late
+    let should_use_old_scroll = true;
     const old_scrollTo = window.scrollTo;
     const next_router_event_target = $6319748b4f0426f0$var$make_next_router_event_target(); // So we can use once: true
     window.scrollTo = (...args)=>{
-        if (should_use_old_scroll || process.env.__NEXT_SCROLL_RESTORATION || !are_on_search_or_category_page()) return Reflect.apply(old_scrollTo, window, args);
+        if (should_use_old_scroll || process.env.__NEXT_SCROLL_RESTORATION || !are_on_search_or_category_page() || history.scrollRestoration === "manual" // I.e. Chimi has manual scroll restoration that we don't want to block https://depictaiworkspace.slack.com/archives/C058KQ3MU8Z/p1686658007164019?thread_ts=1686134246.109689&cid=C058KQ3MU8Z
+        ) return Reflect.apply(old_scrollTo, window, args);
         else should_use_old_scroll = true;
     };
     // When popping state, add padding in the same event as the "popstate" event (required) and then remove it once our stuff is in the DOM

package/dist/module.js

@@ -11,7 +11,7 @@ import {DPC as $TZSbU$DPC} from "@depict-ai/dpc";
 
 /*! Depict.ai search and product listing react SDK wrapper */ // This is where the actual source of your React stuff should live
 // Note, if you add an export here, also add one in src-server/main.ts
-const $c9c4d066f9a5b51b$export$83d89fbfd8236492 = "1.3.1";
+const $c9c4d066f9a5b51b$export$83d89fbfd8236492 = "1.3.3";
 
 
 
@@ -173,11 +173,14 @@ function $8071251ee4e9ffa1$export$6453cbc34515a94d() {
     const are_on_search_or_category_page = ()=>(0, $c83ba9ab79386687$export$775e6b3892b5ad52)() || (0, $c83ba9ab79386687$export$30c8a12df09be0e4)();
     if ($8071251ee4e9ffa1$var$set) return; // Just being safe
     $8071251ee4e9ffa1$var$set = true;
-    let should_use_old_scroll = false;
+    // We used to initialise this variable with false which in some cases (scroll to top button in FloatingButtons) would cause the scroll to be blocked when it shouldn't (see https://depictaiworkspace.slack.com/archives/C058KQ3MU8Z/p1686141321127089?thread_ts=1686134246.109689&cid=C058KQ3MU8Z)
+    // I don't know why we'd want to initialise it with false? The only reason I can see is in the hopes of scroll restoration working on initial page load, but it basically never does due to react rendering and executing our component too late
+    let should_use_old_scroll = true;
     const old_scrollTo = window.scrollTo;
     const next_router_event_target = $8071251ee4e9ffa1$var$make_next_router_event_target(); // So we can use once: true
     window.scrollTo = (...args)=>{
-        if (should_use_old_scroll || process.env.__NEXT_SCROLL_RESTORATION || !are_on_search_or_category_page()) return Reflect.apply(old_scrollTo, window, args);
+        if (should_use_old_scroll || process.env.__NEXT_SCROLL_RESTORATION || !are_on_search_or_category_page() || history.scrollRestoration === "manual" // I.e. Chimi has manual scroll restoration that we don't want to block https://depictaiworkspace.slack.com/archives/C058KQ3MU8Z/p1686658007164019?thread_ts=1686134246.109689&cid=C058KQ3MU8Z
+        ) return Reflect.apply(old_scrollTo, window, args);
         else should_use_old_scroll = true;
     };
     // When popping state, add padding in the same event as the "popstate" event (required) and then remove it once our stuff is in the DOM

package/dist-server/index.d.ts

@@ -33,7 +33,7 @@ interface UseFetchRecommendationsReturn<T extends Display> {
      */
     fetchRecommendations: (opts: FetchRecsOptions) => Promise<T[]>;
 }
-export const version = "1.3.1";
+export const version = "1.3.3";
 export function DepictProvider(props: any): any;
 export function SearchPage(): any;
 export function CategoryPage(): any;

package/dist-server/main.js

@@ -40,7 +40,7 @@ function $35dfe83eec81954c$export$4b564d7f5a1b1d43({ solid_component: solid_comp
 }
 
 
-const $db7b67857900028b$export$83d89fbfd8236492 = "1.3.1";
+const $db7b67857900028b$export$83d89fbfd8236492 = "1.3.3";
 
 
 function $41e7e5d85b5e0d49$export$9fac37a9c21c2f3b(props) {

package/dist-server/module.js

@@ -22,7 +22,7 @@ function $4568c09979447ab1$export$4b564d7f5a1b1d43({ solid_component: solid_comp
 }
 
 
-const $c9c4d066f9a5b51b$export$83d89fbfd8236492 = "1.3.1";
+const $c9c4d066f9a5b51b$export$83d89fbfd8236492 = "1.3.3";
 
 
 function $2b1fdeb83610500f$export$9fac37a9c21c2f3b(props) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@depict-ai/react-ui",
-  "version": "1.3.1",
+  "version": "1.3.3",
   "packageManager": "yarn@3.2.2",
   "source": "src/index.ts",
   "main": "dist/main.js",
@@ -97,9 +97,9 @@
     "README.md"
   ],
   "dependencies": {
-    "@depict-ai/dpc": "^1.0.25",
+    "@depict-ai/dpc": "^1.0.27",
     "@depict-ai/plp-styling": "^1.0.26",
-    "@depict-ai/ui": "^1.3.1",
+    "@depict-ai/ui": "^1.3.2",
     "@depict-ai/utilishared": "^1.0.25",
     "solid-js": "^1.7.3"
   },