@depict-ai/ui 1.0.15 → 1.0.17

package/dist/main.js

@@ -37,7 +37,7 @@ $parcel$export(module.exports, "defaultGridSpacing", () => $a3280f0aad997933$exp
 $parcel$export(module.exports, "defaultColsAtSize", () => $a3280f0aad997933$export$c3834fa86c616831);
 $parcel$export(module.exports, "defaultCategoryTitlePlugin", () => $a3280f0aad997933$export$7bf543b9d9be2c22);
 $parcel$export(module.exports, "version", () => $b757f75811f3261f$export$83d89fbfd8236492);
-/*! Depict.ai search and product listing SDK */ const $b757f75811f3261f$export$83d89fbfd8236492 = "1.0.15";
+/*! Depict.ai search and product listing SDK */ const $b757f75811f3261f$export$83d89fbfd8236492 = "1.0.17";
 
 
 
@@ -821,8 +821,8 @@ function $0caf87eaa19079da$export$f33414a4df0afd3f({ product_card_template_: pro
     });
     return get_elements; // https://github.com/solidjs/solid/releases/tag/v1.7.0
 }
-/**
- * Get one placeholder element. It will be cloned for each placeholder for ultimate performance even when solid-js isn't rendering the placeholders.
+/**
+ * Get one placeholder element. It will be cloned for each placeholder for ultimate performance even when solid-js isn't rendering the placeholders.
  */ function $0caf87eaa19079da$var$Placeholder({ product_card_template_: product_card_template_  }) {
     const get_cloned_placeholders = (element_to_clone = $0caf87eaa19079da$var$cached_placeholder_element)=>Array.isArray(element_to_clone) ? element_to_clone.map((el)=>el instanceof Node ? el.cloneNode(true) : el) : element_to_clone;
     if ($0caf87eaa19079da$var$cached_placeholder_element) return get_cloned_placeholders();
@@ -867,8 +867,8 @@ function $0caf87eaa19079da$export$f33414a4df0afd3f({ product_card_template_: pro
 }
 
 
-/*
- * Usually product templates are somewhat height-elastic depending on the content, i.e. if the title is longer we might up to two extra rows for this. That means that the placeholders almost never will reflect the size of the actual content when we do scroll restoration. This magic number specifies how many extra placeholders we should render everytime to compensate for this, since if the placeholders are in total shorter than the old scroll location the scroll restoration won't work.
+/*
+ * Usually product templates are somewhat height-elastic depending on the content, i.e. if the title is longer we might up to two extra rows for this. That means that the placeholders almost never will reflect the size of the actual content when we do scroll restoration. This magic number specifies how many extra placeholders we should render everytime to compensate for this, since if the placeholders are in total shorter than the old scroll location the scroll restoration won't work.
  */ const $aec24316ed262dc0$export$6c9030e511152266 = 30;
 
 
@@ -960,8 +960,8 @@ function $4117463e2939f5f4$export$63dd0350f8559dda({ displays_by_key_: displays_
 
 
 
-/**
- * Gets product_id (the thing we key the rendering on) from a display
+/**
+ * Gets product_id (the thing we key the rendering on) from a display
  */ function $a5871dacf261adf8$export$548d053aba053d79(display) {
     if ("variant_index" in display) {
         const actual_display = display.variant_displays[display.variant_index];
@@ -995,22 +995,17 @@ function $a73063cc642f3e5f$export$43f0fb951fcb0c54({ displays_by_key_: displays_
             ];
         },
         children: (product_id, index)=>{
-            // We're using `untrack` here since we don't want to react in here to displays_by_key changing
-            // As long as our key stays the same we assume that the display hasn't changed, so we don't want to rerender
-            // If the key gets removed, <For> will remove us
             const owner = (0, $iBTHO$solidjs.getOwner)();
             const display_accessor = (0, $iBTHO$solidjs.createMemo)(()=>displays_by_key_().get(product_id));
-            const [product_card_resource] = (0, $iBTHO$solidjs.createResource)((0, $iBTHO$solidjs.createMemo)(()=>[
-                    merchant_(),
-                    market_()
-                ]), // re-render when merchant or market changes
-            ([merchant, market])=>{
+            const [product_card_resource] = (0, $iBTHO$solidjs.createResource)(display_accessor, // re-render when the reference to the display object changes
+            // this should only be the case when the merchant or market changes and the new request has completed
+            (display)=>{
                 // get display in the function that re-runs when merchant or market have changed, since the display may have changed but product_id not
-                const display = (0, $iBTHO$solidjs.untrack)(display_accessor);
                 return product_card_template_(display, {
                     set_on_index_change: (fn)=>(0, $iBTHO$solidjs.runWithOwner)(owner, ()=>(0, $iBTHO$solidjs.createEffect)(()=>fn(index()))),
-                    merchant: merchant,
-                    market: market
+                    merchant: (0, $iBTHO$solidjs.untrack)(merchant_),
+                    // this means that the merchant and market provided to the template is wrong while we do another fetch, but I don't see how anyone would need to know the correct market there
+                    market: (0, $iBTHO$solidjs.untrack)(market_)
                 });
             });
             const node_accessor = (0, $5f7d55c710fd1bc2$export$2f0cbfbb4354bb52)(product_card_resource);
@@ -1262,9 +1257,9 @@ function $fe76fb463cc60b89$export$3925f87b1545cfc6({ plp_results_: plp_results_
         return _el$;
     })();
 }
-/**
- * Sets styling to styling generated by "Layout" (n_col_styling_with_dist) depending on if filters are open or not
- * This function is special to the actual PLPResults.
+/**
+ * Sets styling to styling generated by "Layout" (n_col_styling_with_dist) depending on if filters are open or not
+ * This function is special to the actual PLPResults.
  */ function $fe76fb463cc60b89$var$results_grid_styling_generator({ layout_options_: layout_options_ , sort_or_filter_open_: sort_or_filter_open_ , set_styling_: set_styling_ , selector_beginning_: selector_beginning_  }) {
     (0, $iBTHO$solidjs.createRenderEffect)(()=>{
         // Add grid styling that changes when filters get opened/closed
@@ -3954,9 +3949,9 @@ function $70482f04605572e2$export$b288c38ae3fddb56({ available_filters_: availab
         last_remote_state = available_filters_();
     });
 }
-/**
- * When a filter is only available locally due to being selected in a previous navigation, we want to make that clear and prevent further "invalid" choices.
- * This sets the counts to 0 (if applicable) and removes non-selected options (if applicable)
+/**
+ * When a filter is only available locally due to being selected in a previous navigation, we want to make that clear and prevent further "invalid" choices.
+ * This sets the counts to 0 (if applicable) and removes non-selected options (if applicable)
  */ function $70482f04605572e2$var$modify_only_local_filter(local_filter_object, selected_filter) {
     const old_meta = local_filter_object.meta;
     const selected_filter_data = selected_filter.data;
@@ -4396,10 +4391,10 @@ const $1ab22f2cd39b3061$var$range_delimiter = "-";
 const $1ab22f2cd39b3061$var$string_item_delimiter = "‚"; // This is not a comma but a comma-like character that browsers will render in the URL bar.
 const $1ab22f2cd39b3061$var$array_item_delimiter = "‣"; // FIXME: decide on a character. Could theoretically be a double string delimiter.
 const $1ab22f2cd39b3061$var$empty_item_replacement = "0️⃣";
-/**
- * Checks if a given array of filter data can be neatly serialized into an URL query parameter, or if it should fall back to JSON+base64.
- * The requirements are either an array of strings, containing no reserved characters, or an array of arrays of strings, containing no reserved characters.
- * No deep or mixed nesting levels are supported.
+/**
+ * Checks if a given array of filter data can be neatly serialized into an URL query parameter, or if it should fall back to JSON+base64.
+ * The requirements are either an array of strings, containing no reserved characters, or an array of arrays of strings, containing no reserved characters.
+ * No deep or mixed nesting levels are supported.
  */ function $1ab22f2cd39b3061$var$array_is_serializable(array, is_root = true) {
     if (array.every((item)=>Array.isArray(item))) {
         if (!is_root) // Deeply nested arrays are not supported, but also don't exist as filters (yet)
@@ -4411,10 +4406,10 @@ const $1ab22f2cd39b3061$var$empty_item_replacement = "0️⃣";
     // Encoding reserved characters is not supported
     return !array.some((item)=>item.includes($1ab22f2cd39b3061$var$array_item_delimiter) || item.includes($1ab22f2cd39b3061$var$string_item_delimiter) || item === $1ab22f2cd39b3061$var$empty_item_replacement);
 }
-/**
- * Human readable string[] or string[][] serialization for URLs.
- * @param array
- * @returns
+/**
+ * Human readable string[] or string[][] serialization for URLs.
+ * @param array
+ * @returns
  */ function $1ab22f2cd39b3061$var$encode_array(array) {
     const is_nested = array.every((item)=>Array.isArray(item));
     if (is_nested) return array.map($1ab22f2cd39b3061$var$encode_array).join($1ab22f2cd39b3061$var$array_item_delimiter) + $1ab22f2cd39b3061$var$array_item_delimiter; // Nested arrays require a suffix to close nesting levels
@@ -5509,8 +5504,8 @@ function $c79fd10a3ab7d6e0$export$477caf01aecaeb2a(options) {
         }
     };
 }
-/**
- * Writes the sorting and filters to the URL query parameters when they change.
+/**
+ * Writes the sorting and filters to the URL query parameters when they change.
  */ function $c79fd10a3ab7d6e0$var$start_href_writer({ get_sorting_: get_sorting_ , get_selected_filters_: get_selected_filters_ , router_: router_ , get_has_sideways_filter_clearing_flag_: get_has_sideways_filter_clearing_flag_  }) {
     (0, $iBTHO$solidjs.createEffect)((0, $iBTHO$solidjs.on)([
         get_sorting_,
@@ -5550,8 +5545,8 @@ function $c79fd10a3ab7d6e0$var$delete_sideways_param_flag(searchParams) {
 
 
 class $e96dd13889871fc0$export$7e1297a3a19f10d8 {
-    /**
-   * Function that will be called when we want to navigate, or a string saying it's not an SPA. You can change this OTG
+    /**
+   * Function that will be called when we want to navigate, or a string saying it's not an SPA. You can change this OTG
    */ #throwing_on_navigation(...args) {
         const { on_navigation_: on_navigation_  } = this;
         if (typeof on_navigation_ !== "function") throw new Error("on_navigation_ is not a function");
@@ -5564,14 +5559,14 @@ class $e96dd13889871fc0$export$7e1297a3a19f10d8 {
             });
         }
     }
-    /**
-   * Checks if this tab shouldn't navigate because user wants to open in new tab
+    /**
+   * Checks if this tab shouldn't navigate because user wants to open in new tab
    */ #dont_attempt_navigation(ev) {
         return ev?.button || ev?.metaKey || ev?.ctrlKey;
     }
-    /**
-   * Vue has async router that needs to be awaited (it navigates after a microtask)
-   * This standardises waiting for all kinds of routers
+    /**
+   * Vue has async router that needs to be awaited (it navigates after a microtask)
+   * This standardises waiting for all kinds of routers
    */ #get_navigation_happened_promise(is_replace) {
         return new Promise((resolve)=>{
             const handler = (what_happened)=>{
@@ -5584,19 +5579,19 @@ class $e96dd13889871fc0$export$7e1297a3a19f10d8 {
         });
     }
     navigate_ = {
-        /**
-     * Call this if you *just* want to replace the state object in history.state
+        /**
+     * Call this if you *just* want to replace the state object in history.state
      */ replace_state_: (new_state)=>{
             // This doesn't break any router, so we just do this directly for now, since many don't expose an interface for changing the state
             history.replaceState(new_state, "");
         },
-        /**
-     * Call this to change the URL
-     * @param new_url_ new URL to go to
-     * @param event_ event if you have this in a click handler, will not do anything on cmd/ctrl or non-left click then
-     * @param force_spa_navigation_ if you want to force SPA navigation even if on_navigation_ is set to "hard_navigation". You should only set this to true for inter-SDK-page navigations
-     * @param is_replace_ if you want to replace the current URL in history instead of pushing a new one
-     * @param category_query_id_ If you are navigating to a category and know the query_id of, please provide it. This is for making a better devEx in js-ui
+        /**
+     * Call this to change the URL
+     * @param new_url_ new URL to go to
+     * @param event_ event if you have this in a click handler, will not do anything on cmd/ctrl or non-left click then
+     * @param force_spa_navigation_ if you want to force SPA navigation even if on_navigation_ is set to "hard_navigation". You should only set this to true for inter-SDK-page navigations
+     * @param is_replace_ if you want to replace the current URL in history instead of pushing a new one
+     * @param category_query_id_ If you are navigating to a category and know the query_id of, please provide it. This is for making a better devEx in js-ui
      */ go_to_: (0, $iBTHO$depictaiutilishared.catchify)(async ({ new_url_: raw_new_url , event_: event_ , force_spa_navigation_: force_spa_navigation_ , is_replace_: is_replace_ , category_query_id_: category_query_id_  })=>{
             const new_url_ = raw_new_url instanceof URL ? raw_new_url : new URL(raw_new_url, location.origin);
             const wants_hard_navigation = this.on_navigation_ === "hard_navigation";
@@ -6404,8 +6399,8 @@ async function $f328bd45f8ede33c$export$e16630ecb9644a1c({ on_submit_with_unchan
 
 
 
-/**
- * Expand an object where every value in the top-level is an accessor into a normal object. Used for converting solid_i18n objects into external i18n typed objects.
+/**
+ * Expand an object where every value in the top-level is an accessor into a normal object. Used for converting solid_i18n objects into external i18n typed objects.
  */ const $639c124c22b6cbaa$export$27b710c08412a7ad = (obj)=>{
     return Object.fromEntries(Object.keys(obj).map((key)=>[
             key,
@@ -7947,8 +7942,8 @@ function $d8e94ea381ab57fa$export$828dcdd425fd3231(aligner) {
 
 
 const $2c61e4cd8229c49a$var$_tmpl$ = /*@__PURE__*/ (0, /*@__PURE__*/$iBTHO$solidjsweb.template)(`<div class="depict plp search"></div>`, 2);
-/**
- * Type for the private properties that are shared between the different exported search SDK wrapper functions. SDK users are not allowed to change/access these, they're an internal implementation detail.
+/**
+ * Type for the private properties that are shared between the different exported search SDK wrapper functions. SDK users are not allowed to change/access these, they're an internal implementation detail.
  */ const $2c61e4cd8229c49a$var$privately_shared_properties = /*@__PURE__*/ new WeakMap();
 const $2c61e4cd8229c49a$var$min_products_to_fetch_ = 20;
 const $2c61e4cd8229c49a$export$bde3eb976dfded9 = (depict_search)=>$2c61e4cd8229c49a$var$privately_shared_properties.get(depict_search)?.search_query_updating_blocked_;
@@ -8345,8 +8340,8 @@ const $9c143d25df1fc2a2$export$d86a33413558014e = (openModalParams)=>{
     }
     const { cleanup: cleanup  } = (0, $d8e94ea381ab57fa$export$828dcdd425fd3231)(openModalParams.element);
     let dispose_aligner;
-    /**
-   * Setting position to fixed is important to make the modal work with align_field.
+    /**
+   * Setting position to fixed is important to make the modal work with align_field.
    */ const style_props_ = (0, $iBTHO$solidjs.createSignal)({
         position: "fixed"
     });
@@ -8442,8 +8437,8 @@ class $0a6acf31d1eae57c$export$3ccaa27d81dd8556 extends EventTarget {
         })();
         return new Set();
     })();
-    /**
-   * Returns if the "replacement page" is currently open
+    /**
+   * Returns if the "replacement page" is currently open
    */ get open_() {
         // make this "readonly" from the outside, the only way to open is by making the url change so that is_page_ returns true
         return this.#is_open;
@@ -8528,26 +8523,26 @@ class $0a6acf31d1eae57c$export$3ccaa27d81dd8556 extends EventTarget {
             if (what_happened !== "popstate") return;
             const cleaned_current_url = $0a6acf31d1eae57c$var$clean_url(location.href);
             if (!$0a6acf31d1eae57c$export$3ccaa27d81dd8556.#visited_urls.has(cleaned_current_url)) {
-                /*
-        What does this check do? Well, there's a bug without it where people end up on a page that they shouldn't see (i.e. a somewhat blank page).
-        The culprit of that bug is: the browser remembers where to make "hard"-navigations (i.e. a full document rebuild with a new `document` and `window` object) when going back and forwards in the history and where to just change the url. What it doesn't always correctly remember (observed in at least chrome and firefox), is *what document to hard-navigate to*. Therefore, we have to jump in and take care of that.
-        The reproduction is this:
-          Spendrups has set up a search page for us under /start/search. We don't want to hard-reload the page though to get there.
-          1. Go to a product page like https://ehandel.spendrups.se/Product/1104201
-          2. Make a search
-          We use PageReplacer to just replace the current page (and the url in the address bar) with the search page. The thing is though, that when going back now the URL just updates but the browser doesn't actually navigate - that's fine, PageReplacer will just restore the old page contents
-          3. Try going back just for fun
-          Everything works as expected
-          4. Go forwards again (just for fun, skip this if you skipped 3)
-          Now the trouble comes when we introduce "hard" navigation.
-          5. Click on a search result. The browser will now "hard" navigate to a new page.
-          6. Click "back"
-          What happens now, is that the browser hard-navigates to /start/search.
-          7. Click "back" again
-          What happens now, is that the browser "soft-navigates" (just changes the URL) to https://ehandel.spendrups.se/Product/1104201
-          PageReplacer unload itself and reveals the original document - it is expected to be the one under https://ehandel.spendrups.se/Product/1104201, but actually it is the one under /start/search - a white page.
-            The fix: is to remember all pages that we have visited since the creation of `window` (we do that by having a static property on PageReplacer since we assume the PageReplacer source code will only exist once on the page and will be loaded at page load, the latter assumption might be wrong, but I hope not). If we get a "popstate" event (meaning back or forwards button, not new page being pushed) to an unknown page we reload.
-             */ (0, $iBTHO$depictaiutilishared.dlog)("PageReplacer: ", cleaned_current_url, " hasn't been visited since document was created, reloading since we won't be able to show this page otherwise");
+                /*
+        What does this check do? Well, there's a bug without it where people end up on a page that they shouldn't see (i.e. a somewhat blank page).
+        The culprit of that bug is: the browser remembers where to make "hard"-navigations (i.e. a full document rebuild with a new `document` and `window` object) when going back and forwards in the history and where to just change the url. What it doesn't always correctly remember (observed in at least chrome and firefox), is *what document to hard-navigate to*. Therefore, we have to jump in and take care of that.
+        The reproduction is this:
+          Spendrups has set up a search page for us under /start/search. We don't want to hard-reload the page though to get there.
+          1. Go to a product page like https://ehandel.spendrups.se/Product/1104201
+          2. Make a search
+          We use PageReplacer to just replace the current page (and the url in the address bar) with the search page. The thing is though, that when going back now the URL just updates but the browser doesn't actually navigate - that's fine, PageReplacer will just restore the old page contents
+          3. Try going back just for fun
+          Everything works as expected
+          4. Go forwards again (just for fun, skip this if you skipped 3)
+          Now the trouble comes when we introduce "hard" navigation.
+          5. Click on a search result. The browser will now "hard" navigate to a new page.
+          6. Click "back"
+          What happens now, is that the browser hard-navigates to /start/search.
+          7. Click "back" again
+          What happens now, is that the browser "soft-navigates" (just changes the URL) to https://ehandel.spendrups.se/Product/1104201
+          PageReplacer unload itself and reveals the original document - it is expected to be the one under https://ehandel.spendrups.se/Product/1104201, but actually it is the one under /start/search - a white page.
+           The fix: is to remember all pages that we have visited since the creation of `window` (we do that by having a static property on PageReplacer since we assume the PageReplacer source code will only exist once on the page and will be loaded at page load, the latter assumption might be wrong, but I hope not). If we get a "popstate" event (meaning back or forwards button, not new page being pushed) to an unknown page we reload.
+            */ (0, $iBTHO$depictaiutilishared.dlog)("PageReplacer: ", cleaned_current_url, " hasn't been visited since document was created, reloading since we won't be able to show this page otherwise");
                 location.reload();
                 $0a6acf31d1eae57c$export$3ccaa27d81dd8556.#reverting_permitted = false; // Disallow any reverting since we can't - it would flash a white page until the reload has finished.
                 this.#unload(); // Call unload to "kill" the solid-js stuff quicker so that it stops updating - the user is expecting the next thing that happens to be a navigation to the actual page the URL says they're on.
@@ -8556,8 +8551,8 @@ class $0a6acf31d1eae57c$export$3ccaa27d81dd8556 extends EventTarget {
         (0, $iBTHO$depictaiutilishared.instant_exec_on_suspect_history_change).add(on_maybe_popstate);
         this.#revert_fns.push(()=>(0, $iBTHO$depictaiutilishared.instant_exec_on_suspect_history_change).delete(on_maybe_popstate)); // Don't run the bugfixer if there isn't a PageReplacer page open since I assume the bug only happens when going back from a PageReplace'd page
     }
-    /**
-   * Stops listening to URL changes and closes the page (if it is currently open)
+    /**
+   * Stops listening to URL changes and closes the page (if it is currently open)
    */ destroy_() {
         this.#unload();
         this.#stop_listening?.();
@@ -8572,8 +8567,8 @@ class $0a6acf31d1eae57c$export$3ccaa27d81dd8556 extends EventTarget {
         if (!is_spa_) (0, $08e7f10b1831645f$export$689e204eadb33baf)(this); // Make so that clicking on links to the previous page work instantly
     }
 }
-/**
- * Remove hash from URL
+/**
+ * Remove hash from URL
  */ function $0a6acf31d1eae57c$var$clean_url(url) {
     // Would want to have this a static private function on PageReplacer, but that's not supported in MobileSafari yet
     // remove hash, all other params *could* trigger a re-request

package/dist/module.js

@@ -2,7 +2,7 @@ import {add_to_version_info as $kyEB6$add_to_version_info, fetch_recs as $kyEB6$
 import {template as $kyEB6$template, insert as $kyEB6$insert, effect as $kyEB6$effect, setAttribute as $kyEB6$setAttribute, createComponent as $kyEB6$createComponent, use as $kyEB6$use, memo as $kyEB6$memo, addEventListener as $kyEB6$addEventListener, delegateEvents as $kyEB6$delegateEvents, spread as $kyEB6$spread, classList as $kyEB6$classList, style as $kyEB6$style, Dynamic as $kyEB6$Dynamic, mergeProps as $kyEB6$mergeProps, className as $kyEB6$className} from "solid-js/web";
 import {createSignal as $kyEB6$createSignal, getOwner as $kyEB6$getOwner, createRoot as $kyEB6$createRoot, runWithOwner as $kyEB6$runWithOwner, onCleanup as $kyEB6$onCleanup, createComputed as $kyEB6$createComputed, createRenderEffect as $kyEB6$createRenderEffect, createEffect as $kyEB6$createEffect, untrack as $kyEB6$untrack, batch as $kyEB6$batch, createResource as $kyEB6$createResource, createMemo as $kyEB6$createMemo, Show as $kyEB6$Show, createReaction as $kyEB6$createReaction, Suspense as $kyEB6$Suspense, on as $kyEB6$on, ErrorBoundary as $kyEB6$ErrorBoundary, For as $kyEB6$For, Index as $kyEB6$Index} from "solid-js";
 
-/*! Depict.ai search and product listing SDK */ const $ec291e0010116877$export$83d89fbfd8236492 = "1.0.15";
+/*! Depict.ai search and product listing SDK */ const $ec291e0010116877$export$83d89fbfd8236492 = "1.0.17";
 
 
 
@@ -786,8 +786,8 @@ function $6e41f1fde994db72$export$f33414a4df0afd3f({ product_card_template_: pro
     });
     return get_elements; // https://github.com/solidjs/solid/releases/tag/v1.7.0
 }
-/**
- * Get one placeholder element. It will be cloned for each placeholder for ultimate performance even when solid-js isn't rendering the placeholders.
+/**
+ * Get one placeholder element. It will be cloned for each placeholder for ultimate performance even when solid-js isn't rendering the placeholders.
  */ function $6e41f1fde994db72$var$Placeholder({ product_card_template_: product_card_template_  }) {
     const get_cloned_placeholders = (element_to_clone = $6e41f1fde994db72$var$cached_placeholder_element)=>Array.isArray(element_to_clone) ? element_to_clone.map((el)=>el instanceof Node ? el.cloneNode(true) : el) : element_to_clone;
     if ($6e41f1fde994db72$var$cached_placeholder_element) return get_cloned_placeholders();
@@ -832,8 +832,8 @@ function $6e41f1fde994db72$export$f33414a4df0afd3f({ product_card_template_: pro
 }
 
 
-/*
- * Usually product templates are somewhat height-elastic depending on the content, i.e. if the title is longer we might up to two extra rows for this. That means that the placeholders almost never will reflect the size of the actual content when we do scroll restoration. This magic number specifies how many extra placeholders we should render everytime to compensate for this, since if the placeholders are in total shorter than the old scroll location the scroll restoration won't work.
+/*
+ * Usually product templates are somewhat height-elastic depending on the content, i.e. if the title is longer we might up to two extra rows for this. That means that the placeholders almost never will reflect the size of the actual content when we do scroll restoration. This magic number specifies how many extra placeholders we should render everytime to compensate for this, since if the placeholders are in total shorter than the old scroll location the scroll restoration won't work.
  */ const $7747fe1a8c5652e3$export$6c9030e511152266 = 30;
 
 
@@ -925,8 +925,8 @@ function $0e0b96a8dc1ad492$export$63dd0350f8559dda({ displays_by_key_: displays_
 
 
 
-/**
- * Gets product_id (the thing we key the rendering on) from a display
+/**
+ * Gets product_id (the thing we key the rendering on) from a display
  */ function $16351e4c87b90b3d$export$548d053aba053d79(display) {
     if ("variant_index" in display) {
         const actual_display = display.variant_displays[display.variant_index];
@@ -960,22 +960,17 @@ function $29ec7cddbe1d3484$export$43f0fb951fcb0c54({ displays_by_key_: displays_
             ];
         },
         children: (product_id, index)=>{
-            // We're using `untrack` here since we don't want to react in here to displays_by_key changing
-            // As long as our key stays the same we assume that the display hasn't changed, so we don't want to rerender
-            // If the key gets removed, <For> will remove us
             const owner = (0, $kyEB6$getOwner)();
             const display_accessor = (0, $kyEB6$createMemo)(()=>displays_by_key_().get(product_id));
-            const [product_card_resource] = (0, $kyEB6$createResource)((0, $kyEB6$createMemo)(()=>[
-                    merchant_(),
-                    market_()
-                ]), // re-render when merchant or market changes
-            ([merchant, market])=>{
+            const [product_card_resource] = (0, $kyEB6$createResource)(display_accessor, // re-render when the reference to the display object changes
+            // this should only be the case when the merchant or market changes and the new request has completed
+            (display)=>{
                 // get display in the function that re-runs when merchant or market have changed, since the display may have changed but product_id not
-                const display = (0, $kyEB6$untrack)(display_accessor);
                 return product_card_template_(display, {
                     set_on_index_change: (fn)=>(0, $kyEB6$runWithOwner)(owner, ()=>(0, $kyEB6$createEffect)(()=>fn(index()))),
-                    merchant: merchant,
-                    market: market
+                    merchant: (0, $kyEB6$untrack)(merchant_),
+                    // this means that the merchant and market provided to the template is wrong while we do another fetch, but I don't see how anyone would need to know the correct market there
+                    market: (0, $kyEB6$untrack)(market_)
                 });
             });
             const node_accessor = (0, $a52235d12ccf9c88$export$2f0cbfbb4354bb52)(product_card_resource);
@@ -1227,9 +1222,9 @@ function $2d25e42846eac370$export$3925f87b1545cfc6({ plp_results_: plp_results_
         return _el$;
     })();
 }
-/**
- * Sets styling to styling generated by "Layout" (n_col_styling_with_dist) depending on if filters are open or not
- * This function is special to the actual PLPResults.
+/**
+ * Sets styling to styling generated by "Layout" (n_col_styling_with_dist) depending on if filters are open or not
+ * This function is special to the actual PLPResults.
  */ function $2d25e42846eac370$var$results_grid_styling_generator({ layout_options_: layout_options_ , sort_or_filter_open_: sort_or_filter_open_ , set_styling_: set_styling_ , selector_beginning_: selector_beginning_  }) {
     (0, $kyEB6$createRenderEffect)(()=>{
         // Add grid styling that changes when filters get opened/closed
@@ -3919,9 +3914,9 @@ function $5efb5894079b220e$export$b288c38ae3fddb56({ available_filters_: availab
         last_remote_state = available_filters_();
     });
 }
-/**
- * When a filter is only available locally due to being selected in a previous navigation, we want to make that clear and prevent further "invalid" choices.
- * This sets the counts to 0 (if applicable) and removes non-selected options (if applicable)
+/**
+ * When a filter is only available locally due to being selected in a previous navigation, we want to make that clear and prevent further "invalid" choices.
+ * This sets the counts to 0 (if applicable) and removes non-selected options (if applicable)
  */ function $5efb5894079b220e$var$modify_only_local_filter(local_filter_object, selected_filter) {
     const old_meta = local_filter_object.meta;
     const selected_filter_data = selected_filter.data;
@@ -4361,10 +4356,10 @@ const $4147651646da990a$var$range_delimiter = "-";
 const $4147651646da990a$var$string_item_delimiter = "‚"; // This is not a comma but a comma-like character that browsers will render in the URL bar.
 const $4147651646da990a$var$array_item_delimiter = "‣"; // FIXME: decide on a character. Could theoretically be a double string delimiter.
 const $4147651646da990a$var$empty_item_replacement = "0️⃣";
-/**
- * Checks if a given array of filter data can be neatly serialized into an URL query parameter, or if it should fall back to JSON+base64.
- * The requirements are either an array of strings, containing no reserved characters, or an array of arrays of strings, containing no reserved characters.
- * No deep or mixed nesting levels are supported.
+/**
+ * Checks if a given array of filter data can be neatly serialized into an URL query parameter, or if it should fall back to JSON+base64.
+ * The requirements are either an array of strings, containing no reserved characters, or an array of arrays of strings, containing no reserved characters.
+ * No deep or mixed nesting levels are supported.
  */ function $4147651646da990a$var$array_is_serializable(array, is_root = true) {
     if (array.every((item)=>Array.isArray(item))) {
         if (!is_root) // Deeply nested arrays are not supported, but also don't exist as filters (yet)
@@ -4376,10 +4371,10 @@ const $4147651646da990a$var$empty_item_replacement = "0️⃣";
     // Encoding reserved characters is not supported
     return !array.some((item)=>item.includes($4147651646da990a$var$array_item_delimiter) || item.includes($4147651646da990a$var$string_item_delimiter) || item === $4147651646da990a$var$empty_item_replacement);
 }
-/**
- * Human readable string[] or string[][] serialization for URLs.
- * @param array
- * @returns
+/**
+ * Human readable string[] or string[][] serialization for URLs.
+ * @param array
+ * @returns
  */ function $4147651646da990a$var$encode_array(array) {
     const is_nested = array.every((item)=>Array.isArray(item));
     if (is_nested) return array.map($4147651646da990a$var$encode_array).join($4147651646da990a$var$array_item_delimiter) + $4147651646da990a$var$array_item_delimiter; // Nested arrays require a suffix to close nesting levels
@@ -5474,8 +5469,8 @@ function $ccd5db07a3a069c5$export$477caf01aecaeb2a(options) {
         }
     };
 }
-/**
- * Writes the sorting and filters to the URL query parameters when they change.
+/**
+ * Writes the sorting and filters to the URL query parameters when they change.
  */ function $ccd5db07a3a069c5$var$start_href_writer({ get_sorting_: get_sorting_ , get_selected_filters_: get_selected_filters_ , router_: router_ , get_has_sideways_filter_clearing_flag_: get_has_sideways_filter_clearing_flag_  }) {
     (0, $kyEB6$createEffect)((0, $kyEB6$on)([
         get_sorting_,
@@ -5515,8 +5510,8 @@ function $ccd5db07a3a069c5$var$delete_sideways_param_flag(searchParams) {
 
 
 class $4fd2ba03703a7d92$export$7e1297a3a19f10d8 {
-    /**
-   * Function that will be called when we want to navigate, or a string saying it's not an SPA. You can change this OTG
+    /**
+   * Function that will be called when we want to navigate, or a string saying it's not an SPA. You can change this OTG
    */ #throwing_on_navigation(...args) {
         const { on_navigation_: on_navigation_  } = this;
         if (typeof on_navigation_ !== "function") throw new Error("on_navigation_ is not a function");
@@ -5529,14 +5524,14 @@ class $4fd2ba03703a7d92$export$7e1297a3a19f10d8 {
             });
         }
     }
-    /**
-   * Checks if this tab shouldn't navigate because user wants to open in new tab
+    /**
+   * Checks if this tab shouldn't navigate because user wants to open in new tab
    */ #dont_attempt_navigation(ev) {
         return ev?.button || ev?.metaKey || ev?.ctrlKey;
     }
-    /**
-   * Vue has async router that needs to be awaited (it navigates after a microtask)
-   * This standardises waiting for all kinds of routers
+    /**
+   * Vue has async router that needs to be awaited (it navigates after a microtask)
+   * This standardises waiting for all kinds of routers
    */ #get_navigation_happened_promise(is_replace) {
         return new Promise((resolve)=>{
             const handler = (what_happened)=>{
@@ -5549,19 +5544,19 @@ class $4fd2ba03703a7d92$export$7e1297a3a19f10d8 {
         });
     }
     navigate_ = {
-        /**
-     * Call this if you *just* want to replace the state object in history.state
+        /**
+     * Call this if you *just* want to replace the state object in history.state
      */ replace_state_: (new_state)=>{
             // This doesn't break any router, so we just do this directly for now, since many don't expose an interface for changing the state
             history.replaceState(new_state, "");
         },
-        /**
-     * Call this to change the URL
-     * @param new_url_ new URL to go to
-     * @param event_ event if you have this in a click handler, will not do anything on cmd/ctrl or non-left click then
-     * @param force_spa_navigation_ if you want to force SPA navigation even if on_navigation_ is set to "hard_navigation". You should only set this to true for inter-SDK-page navigations
-     * @param is_replace_ if you want to replace the current URL in history instead of pushing a new one
-     * @param category_query_id_ If you are navigating to a category and know the query_id of, please provide it. This is for making a better devEx in js-ui
+        /**
+     * Call this to change the URL
+     * @param new_url_ new URL to go to
+     * @param event_ event if you have this in a click handler, will not do anything on cmd/ctrl or non-left click then
+     * @param force_spa_navigation_ if you want to force SPA navigation even if on_navigation_ is set to "hard_navigation". You should only set this to true for inter-SDK-page navigations
+     * @param is_replace_ if you want to replace the current URL in history instead of pushing a new one
+     * @param category_query_id_ If you are navigating to a category and know the query_id of, please provide it. This is for making a better devEx in js-ui
      */ go_to_: (0, $kyEB6$catchify)(async ({ new_url_: raw_new_url , event_: event_ , force_spa_navigation_: force_spa_navigation_ , is_replace_: is_replace_ , category_query_id_: category_query_id_  })=>{
             const new_url_ = raw_new_url instanceof URL ? raw_new_url : new URL(raw_new_url, location.origin);
             const wants_hard_navigation = this.on_navigation_ === "hard_navigation";
@@ -6369,8 +6364,8 @@ async function $bfd908d13d90934c$export$e16630ecb9644a1c({ on_submit_with_unchan
 
 
 
-/**
- * Expand an object where every value in the top-level is an accessor into a normal object. Used for converting solid_i18n objects into external i18n typed objects.
+/**
+ * Expand an object where every value in the top-level is an accessor into a normal object. Used for converting solid_i18n objects into external i18n typed objects.
  */ const $c97b1f0e44afb9d2$export$27b710c08412a7ad = (obj)=>{
     return Object.fromEntries(Object.keys(obj).map((key)=>[
             key,
@@ -7912,8 +7907,8 @@ function $4ca91518c9a35379$export$828dcdd425fd3231(aligner) {
 
 
 const $806c97b931758897$var$_tmpl$ = /*@__PURE__*/ (0, /*@__PURE__*/$kyEB6$template)(`<div class="depict plp search"></div>`, 2);
-/**
- * Type for the private properties that are shared between the different exported search SDK wrapper functions. SDK users are not allowed to change/access these, they're an internal implementation detail.
+/**
+ * Type for the private properties that are shared between the different exported search SDK wrapper functions. SDK users are not allowed to change/access these, they're an internal implementation detail.
  */ const $806c97b931758897$var$privately_shared_properties = /*@__PURE__*/ new WeakMap();
 const $806c97b931758897$var$min_products_to_fetch_ = 20;
 const $806c97b931758897$export$bde3eb976dfded9 = (depict_search)=>$806c97b931758897$var$privately_shared_properties.get(depict_search)?.search_query_updating_blocked_;
@@ -8310,8 +8305,8 @@ const $6891c018a834cfdd$export$d86a33413558014e = (openModalParams)=>{
     }
     const { cleanup: cleanup  } = (0, $4ca91518c9a35379$export$828dcdd425fd3231)(openModalParams.element);
     let dispose_aligner;
-    /**
-   * Setting position to fixed is important to make the modal work with align_field.
+    /**
+   * Setting position to fixed is important to make the modal work with align_field.
    */ const style_props_ = (0, $kyEB6$createSignal)({
         position: "fixed"
     });
@@ -8407,8 +8402,8 @@ class $753e9b2889a47c89$export$3ccaa27d81dd8556 extends EventTarget {
         })();
         return new Set();
     })();
-    /**
-   * Returns if the "replacement page" is currently open
+    /**
+   * Returns if the "replacement page" is currently open
    */ get open_() {
         // make this "readonly" from the outside, the only way to open is by making the url change so that is_page_ returns true
         return this.#is_open;
@@ -8493,26 +8488,26 @@ class $753e9b2889a47c89$export$3ccaa27d81dd8556 extends EventTarget {
             if (what_happened !== "popstate") return;
             const cleaned_current_url = $753e9b2889a47c89$var$clean_url(location.href);
             if (!$753e9b2889a47c89$export$3ccaa27d81dd8556.#visited_urls.has(cleaned_current_url)) {
-                /*
-        What does this check do? Well, there's a bug without it where people end up on a page that they shouldn't see (i.e. a somewhat blank page).
-        The culprit of that bug is: the browser remembers where to make "hard"-navigations (i.e. a full document rebuild with a new `document` and `window` object) when going back and forwards in the history and where to just change the url. What it doesn't always correctly remember (observed in at least chrome and firefox), is *what document to hard-navigate to*. Therefore, we have to jump in and take care of that.
-        The reproduction is this:
-          Spendrups has set up a search page for us under /start/search. We don't want to hard-reload the page though to get there.
-          1. Go to a product page like https://ehandel.spendrups.se/Product/1104201
-          2. Make a search
-          We use PageReplacer to just replace the current page (and the url in the address bar) with the search page. The thing is though, that when going back now the URL just updates but the browser doesn't actually navigate - that's fine, PageReplacer will just restore the old page contents
-          3. Try going back just for fun
-          Everything works as expected
-          4. Go forwards again (just for fun, skip this if you skipped 3)
-          Now the trouble comes when we introduce "hard" navigation.
-          5. Click on a search result. The browser will now "hard" navigate to a new page.
-          6. Click "back"
-          What happens now, is that the browser hard-navigates to /start/search.
-          7. Click "back" again
-          What happens now, is that the browser "soft-navigates" (just changes the URL) to https://ehandel.spendrups.se/Product/1104201
-          PageReplacer unload itself and reveals the original document - it is expected to be the one under https://ehandel.spendrups.se/Product/1104201, but actually it is the one under /start/search - a white page.
-            The fix: is to remember all pages that we have visited since the creation of `window` (we do that by having a static property on PageReplacer since we assume the PageReplacer source code will only exist once on the page and will be loaded at page load, the latter assumption might be wrong, but I hope not). If we get a "popstate" event (meaning back or forwards button, not new page being pushed) to an unknown page we reload.
-             */ (0, $kyEB6$dlog)("PageReplacer: ", cleaned_current_url, " hasn't been visited since document was created, reloading since we won't be able to show this page otherwise");
+                /*
+        What does this check do? Well, there's a bug without it where people end up on a page that they shouldn't see (i.e. a somewhat blank page).
+        The culprit of that bug is: the browser remembers where to make "hard"-navigations (i.e. a full document rebuild with a new `document` and `window` object) when going back and forwards in the history and where to just change the url. What it doesn't always correctly remember (observed in at least chrome and firefox), is *what document to hard-navigate to*. Therefore, we have to jump in and take care of that.
+        The reproduction is this:
+          Spendrups has set up a search page for us under /start/search. We don't want to hard-reload the page though to get there.
+          1. Go to a product page like https://ehandel.spendrups.se/Product/1104201
+          2. Make a search
+          We use PageReplacer to just replace the current page (and the url in the address bar) with the search page. The thing is though, that when going back now the URL just updates but the browser doesn't actually navigate - that's fine, PageReplacer will just restore the old page contents
+          3. Try going back just for fun
+          Everything works as expected
+          4. Go forwards again (just for fun, skip this if you skipped 3)
+          Now the trouble comes when we introduce "hard" navigation.
+          5. Click on a search result. The browser will now "hard" navigate to a new page.
+          6. Click "back"
+          What happens now, is that the browser hard-navigates to /start/search.
+          7. Click "back" again
+          What happens now, is that the browser "soft-navigates" (just changes the URL) to https://ehandel.spendrups.se/Product/1104201
+          PageReplacer unload itself and reveals the original document - it is expected to be the one under https://ehandel.spendrups.se/Product/1104201, but actually it is the one under /start/search - a white page.
+           The fix: is to remember all pages that we have visited since the creation of `window` (we do that by having a static property on PageReplacer since we assume the PageReplacer source code will only exist once on the page and will be loaded at page load, the latter assumption might be wrong, but I hope not). If we get a "popstate" event (meaning back or forwards button, not new page being pushed) to an unknown page we reload.
+            */ (0, $kyEB6$dlog)("PageReplacer: ", cleaned_current_url, " hasn't been visited since document was created, reloading since we won't be able to show this page otherwise");
                 location.reload();
                 $753e9b2889a47c89$export$3ccaa27d81dd8556.#reverting_permitted = false; // Disallow any reverting since we can't - it would flash a white page until the reload has finished.
                 this.#unload(); // Call unload to "kill" the solid-js stuff quicker so that it stops updating - the user is expecting the next thing that happens to be a navigation to the actual page the URL says they're on.
@@ -8521,8 +8516,8 @@ class $753e9b2889a47c89$export$3ccaa27d81dd8556 extends EventTarget {
         (0, $kyEB6$instant_exec_on_suspect_history_change).add(on_maybe_popstate);
         this.#revert_fns.push(()=>(0, $kyEB6$instant_exec_on_suspect_history_change).delete(on_maybe_popstate)); // Don't run the bugfixer if there isn't a PageReplacer page open since I assume the bug only happens when going back from a PageReplace'd page
     }
-    /**
-   * Stops listening to URL changes and closes the page (if it is currently open)
+    /**
+   * Stops listening to URL changes and closes the page (if it is currently open)
    */ destroy_() {
         this.#unload();
         this.#stop_listening?.();
@@ -8537,8 +8532,8 @@ class $753e9b2889a47c89$export$3ccaa27d81dd8556 extends EventTarget {
         if (!is_spa_) (0, $6936724b9e0b98d5$export$689e204eadb33baf)(this); // Make so that clicking on links to the previous page work instantly
     }
 }
-/**
- * Remove hash from URL
+/**
+ * Remove hash from URL
  */ function $753e9b2889a47c89$var$clean_url(url) {
     // Would want to have this a static private function on PageReplacer, but that's not supported in MobileSafari yet
     // remove hash, all other params *could* trigger a re-request