diff --git a/e2e/dashboard.spec.ts b/e2e/dashboard.spec.ts index ecb108e..e481cc2 100644 --- a/e2e/dashboard.spec.ts +++ b/e2e/dashboard.spec.ts @@ -59,19 +59,16 @@ test.describe("Dashboard", () => { // The old "collection card with item count of 6" test referenced a dashboard card // that no longer exists post-Phase-27. Mark as fixme until discovery feed is seeded. - test.fixme( - "shows collection card with item count of 6", - async ({ page }) => { - // NOTE: The old Collection dashboard card is removed. The landing page now - // shows discovery sections (Popular Setups, Recently Added, etc.). - // This test needs to be replaced with a discovery-feed-aware assertion. - const collectionCard = page - .getByRole("link", { name: /collection/i }) - .first(); - await expect(collectionCard).toBeVisible(); - await expect(collectionCard.getByText("6")).toBeVisible(); - }, - ); + test.fixme("shows collection card with item count of 6", async ({ page }) => { + // NOTE: The old Collection dashboard card is removed. The landing page now + // shows discovery sections (Popular Setups, Recently Added, etc.). + // This test needs to be replaced with a discovery-feed-aware assertion. + const collectionCard = page + .getByRole("link", { name: /collection/i }) + .first(); + await expect(collectionCard).toBeVisible(); + await expect(collectionCard.getByText("6")).toBeVisible(); + }); // Planning card removed from dashboard — threads are accessed via Collection > Planning tab test.fixme("shows active thread count on Planning card", async ({ page }) => { diff --git a/src/client/components/BottomTabBar.tsx b/src/client/components/BottomTabBar.tsx index 8497f39..77d841b 100644 --- a/src/client/components/BottomTabBar.tsx +++ b/src/client/components/BottomTabBar.tsx @@ -10,18 +10,15 @@ interface TabItemProps { isActive: boolean; } -function TabItemWrapper({ - icon, - label, - isActive, - children, -}: TabItemProps & { children?: React.ReactNode }) { +function TabItemWrapper({ icon, label, isActive }: TabItemProps) { const activeClass = "text-gray-900"; const inactiveClass = "text-gray-400"; const colorClass = isActive ? activeClass : inactiveClass; return ( - + {label} @@ -83,10 +80,7 @@ export function BottomTabBar() { )} {/* Search tab — always a button, opens CatalogSearchOverlay */} - diff --git a/src/client/routes/__root.tsx b/src/client/routes/__root.tsx index e3ca195..97a8ef7 100644 --- a/src/client/routes/__root.tsx +++ b/src/client/routes/__root.tsx @@ -13,10 +13,10 @@ import "../app.css"; import { AddToCollectionModal } from "../components/AddToCollectionModal"; import { AddToThreadModal } from "../components/AddToThreadModal"; import { AuthPromptModal } from "../components/AuthPromptModal"; +import { BottomTabBar } from "../components/BottomTabBar"; import { CatalogSearchOverlay } from "../components/CatalogSearchOverlay"; import { ConfirmDialog } from "../components/ConfirmDialog"; import { ExternalLinkDialog } from "../components/ExternalLinkDialog"; -import { BottomTabBar } from "../components/BottomTabBar"; import { FabMenu } from "../components/FabMenu"; import { OnboardingWizard } from "../components/OnboardingWizard"; import { TopNav } from "../components/TopNav"; diff --git a/src/client/routes/collection/index.tsx b/src/client/routes/collection/index.tsx index 4a51a68..f2d60fa 100644 --- a/src/client/routes/collection/index.tsx +++ b/src/client/routes/collection/index.tsx @@ -66,11 +66,7 @@ function CollectionPage() { exit="exit" transition={{ duration: 0.12, ease: "easeInOut" }} > - {tab === "gear" ? ( - - ) : ( - - )} + {tab === "gear" ? : }