From ac46c1504f128d50ddc24d8dd828cee0300346ed Mon Sep 17 00:00:00 2001 From: jamey Date: Wed, 11 Feb 2026 00:32:15 +0000 Subject: [PATCH] remove redundant "view basket" link from cart drawer MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The drawer now has full quantity controls, remove, subtotal, and checkout — the link to the cart page added friction without value. Cart page remains accessible via the basket icon in nav. Co-Authored-By: Claude Opus 4.6 --- PROGRESS.md | 3 ++- .../components/shop_components/cart.ex | 22 ------------------- 2 files changed, 2 insertions(+), 23 deletions(-) diff --git a/PROGRESS.md b/PROGRESS.md index 61167a5..28f5284 100644 --- a/PROGRESS.md +++ b/PROGRESS.md @@ -49,7 +49,8 @@ Issues found during hands-on testing of the deployed prod site on mobile and des ### Cart - [x] Should be able to change quantity in the cart drawer (currently only on cart page?) -- [ ] Cart drawer button → "view basket" feels redundant — streamline the flow +- [x] Cart drawer button → "view basket" feels redundant — streamline the flow +- [ ] Shipping costs: add Stripe shipping options or query Printify for dynamic rates ### Navigation & links - [ ] Search doesn't work (modal opens but no results/functionality) diff --git a/lib/simpleshop_theme_web/components/shop_components/cart.ex b/lib/simpleshop_theme_web/components/shop_components/cart.ex index 58138b4..936521b 100644 --- a/lib/simpleshop_theme_web/components/shop_components/cart.ex +++ b/lib/simpleshop_theme_web/components/shop_components/cart.ex @@ -147,28 +147,6 @@ defmodule SimpleshopThemeWeb.ShopComponents.Cart do <% end %> - <%= if @mode == :preview do %> - Phoenix.LiveView.JS.push("change_preview_page", value: %{page: "cart"}) - } - class="cart-drawer-link" - style="display: block; text-align: center; font-family: var(--t-font-body); font-size: var(--t-text-small); color: var(--t-text-secondary); text-decoration: underline; cursor: pointer;" - > - View basket - - <% else %> - - View basket - - <% end %> """