jamey/simpleshop_theme
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

drop redundant "Home" from PDP breadcrumbs

Browse Source 
Logo already links home; breadcrumb now starts at category.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
jamey 2026-02-10 15:59:34 +00:00
parent dc8bf28892
commit 8775c2eeef
2 changed files with 13 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/simpleshop_theme_web/components/page_templates/pdp.html.heex
View File

@ -13,9 +13,6 @@
<main id="main-content" class="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8 py-8">
<.breadcrumb
items={
[
%{label: "Home", page: "home", href: "/"}
] ++
if @product.category do
[
%{

9
test/simpleshop_theme_web/live/shop_live/product_show_test.exs
View File

@ -27,19 +27,12 @@ defmodule SimpleshopThemeWeb.ShopLive.ProductShowTest do
assert html =~ SimpleshopTheme.Cart.format_price(product.price)
end
test "renders breadcrumb with Home link", %{conn: conn} do
product = List.first(PreviewData.products())
{:ok, _view, html} = live(conn, ~p"/products/#{product.id}")
assert html =~ "Home"
assert html =~ ~s(href="/")
end
test "renders breadcrumb with category link", %{conn: conn} do
product = List.first(PreviewData.products())
{:ok, _view, html} = live(conn, ~p"/products/#{product.id}")
assert html =~ product.category
assert html =~ "/collections/"
end
test "renders add to cart button", %{conn: conn} do