@metron-labs/quote-react 0.1.32 → 0.1.33

package/dist/app/QuickQuote.js

@@ -97,7 +97,10 @@ function QuickQuoteInner({ fullPage = false, hideHeader = false, internal = fals
         // Our own state only - copying the host's (e.g. Turbolinks' restoration id) makes its
         // router treat back/forward between our steps as its page visits and re-render the page,
         // wiping the quote. Next.js's patched pushState adds its own fields back.
-        window.history.pushState({ boqQuoteStage: next }, "", url);
+        // New results while the URL still carries one of our steps (start over, or the host
+        // re-rendered the page under us) reuse that entry, or back would land on a stale copy.
+        const stale = next === "results" && (window.location.hash === "#matched-products" || window.location.hash === "#quote-request");
+        window.history[stale ? "replaceState" : "pushState"]({ boqQuoteStage: next }, "", url);
     }, []);
     useEffect(() => {
         const onPopState = () => {
@@ -127,11 +130,10 @@ function QuickQuoteInner({ fullPage = false, hideHeader = false, internal = fals
         setStage("basket");
     }, [writeQuoteHistory]);
     const backToResults = useCallback(() => {
+        // Step back over the basket entry goToBasket pushed (popstate picks the screen), rather
+        // than rewriting it: each rewrite left an extra results entry the browser back had to cross.
         if (window.location.hash === "#quote-request") {
-            const url = new URL(window.location.href);
-            url.hash = "matched-products";
-            window.history.replaceState({ boqQuoteStage: "results" }, "", url);
-            setStage("results");
+            window.history.back();
             return;
         }
         setStage(scheduleResult ? "results" : "entry");