From 11635eaeadfb077f3a55ce7374da54eae3ed7a6f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ullrich=20Sch=C3=A4fer?= Date: Sun, 26 Apr 2026 08:44:08 +0200 Subject: [PATCH] Drop redundant "Feed" button on the logged-in home page MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The navbar already has a "Feed" entry that signed-in users see on every page, including /. The page-header button next to "New Activity" duplicated that path without adding anything — they pointed at the same URL, with the navbar entry being the more discoverable of the two. Stream D from docs/information-architecture.md. - apps/journal/app/routes/home.tsx — remove the Feed anchor; "New Activity" stays as the only header CTA on the personal dashboard. - openspec/specs/journal-landing/spec.md — retire the "Social feed link for signed-in users" requirement that prescribed the now-deleted button. Co-Authored-By: Claude Opus 4.7 (1M context) --- apps/journal/app/routes/home.tsx | 20 ++++++-------------- openspec/specs/journal-landing/spec.md | 12 ------------ 2 files changed, 6 insertions(+), 26 deletions(-) diff --git a/apps/journal/app/routes/home.tsx b/apps/journal/app/routes/home.tsx index f03cffa..a1850aa 100644 --- a/apps/journal/app/routes/home.tsx +++ b/apps/journal/app/routes/home.tsx @@ -167,20 +167,12 @@ export default function Home({ loaderData }: Route.ComponentProps) { {user.displayName ?? user.username} -
- - {t("social.feed.title")} - - - {t("activities.new")} - -
+ + {t("activities.new")} + {showAddPasskey && !passkeyDone && supportsPasskey === true && ( diff --git a/openspec/specs/journal-landing/spec.md b/openspec/specs/journal-landing/spec.md index c8cea3e..ea9fd2f 100644 --- a/openspec/specs/journal-landing/spec.md +++ b/openspec/specs/journal-landing/spec.md @@ -53,18 +53,6 @@ For signed-in users, the home page SHALL render a personal activity dashboard: a - **WHEN** a signed-in user has no activities yet - **THEN** the dashboard shows an empty-state message pointing to activity creation, while the welcome line and New Activity CTA remain visible -### Requirement: Social feed link for signed-in users -For signed-in users, the personal dashboard SHALL include a prominent link to the social feed at `/feed` (see `social-follows` spec, "Social activity feed") alongside the existing "New Activity" CTA. The link SHALL be visible regardless of whether the user follows anyone yet — the social feed's own empty state handles the zero-follows case. - -#### Scenario: Feed link on personal dashboard -- **WHEN** a signed-in user loads `/` -- **THEN** the dashboard header shows a "Feed" (or equivalent) link to `/feed` next to the "New Activity" CTA - -#### Scenario: Feed link is not shown to signed-out visitors -- **WHEN** an unauthenticated visitor loads `/` -- **THEN** the visitor-home layout does not expose a link to `/feed` (the route requires authentication) - - ### Requirement: Notifications entry in the navbar The navbar SHALL render a single inbox entry — a bell icon — for signed-in users, linking to `/notifications`. It SHALL render an unread count badge when the user has at least one unread notification, and no badge when the count is zero. The badge live-updates via `sse-broker`; the loader-driven count is the SSR baseline. Follow-request actions (Approve / Reject) live as the Requests tab inside `/notifications` (see `notifications` spec); the navbar SHALL NOT render a separate "Follow requests" entry — the bell is the single inbox surface.