diff --git a/CHECKPOINT-encryption.md b/CHECKPOINT-encryption.md index 643445a..fad8a95 100644 --- a/CHECKPOINT-encryption.md +++ b/CHECKPOINT-encryption.md @@ -6,11 +6,12 @@ verified green at the moment this file was written. ## Where things are - Project: `/home/avi/Projects/Nostr_Keynctr` -- Git repo: `master` @ `835b0a0` ("frontend: list all profiles on Home, pin - dev host, widen profiles card"). Before it: `71e580f` (Trim white margins - from sidebar logo artwork), `f29d4f9` (Checkpoint 0.45 migration), - `fa5ba08` (Security: upgrade nostr stack 0.40->0.45), `bf10ae3`, `e210f17`, - `7098e75`, `e6a1efc`, `6d5063d`, `1c428a9`. +- Git repo: `master` @ `f534165` ("frontend: remove redundant Profiles sidebar + tab"). Before it: `835b0a0` (list all profiles on Home, pin dev host, + widen card), `5635f79` (checkpoint), `71e580f` (Trim white margins), + `f29d4f9` (Checkpoint 0.45 migration), `fa5ba08` (Security: upgrade nostr + stack 0.40->0.45), `bf10ae3`, `e210f17`, `7098e75`, `e6a1efc`, `6d5063d`, + `1c428a9`. - Working tree: still DIRTY with the long-standing user changes — `concept3.svg` deleted, `KeynectrAppIconPossibility02.jpeg` untracked. These are NOT part of the frontend commit above and remain uncommitted. @@ -42,6 +43,7 @@ verified green at the moment this file was written. ## Commits added most recently +- `f534165` frontend: remove redundant Profiles sidebar tab - `835b0a0` frontend: list all profiles on Home, pin dev host, widen profiles card - `fa5ba08` Security: upgrade nostr stack 0.40->0.45 clearing 11 RustSec advisories @@ -1387,6 +1389,12 @@ A stopping point you can return to if this session is closed. half. Changed it to a single `1fr` column and dropped the now-redundant `@media (max-width: 900px)` override, so "Your profiles" matches the width of "Most recent publication" below it. No JSX change. +6. **Removed the "Profiles" sidebar tab.** It was redundant with Home's + "Manage profiles" button (which already navigates to the Profiles screen, + still rendered by `App.tsx`). Deleted the `profiles` entry from `NAV_ITEMS` + in `Sidebar.tsx`; the Profiles screen, route, and Manage button are all + unchanged. Updated `App.test.tsx` to reach the screen via "Manage profiles" + instead of the removed tab. ### Files changed diff --git a/frontend/src/components/Sidebar.tsx b/frontend/src/components/Sidebar.tsx index cce4bca..5a4aab7 100644 --- a/frontend/src/components/Sidebar.tsx +++ b/frontend/src/components/Sidebar.tsx @@ -9,7 +9,6 @@ import logoUrl from '../assets/logo.png'; const NAV_ITEMS: { id: Screen; label: string; icon: IconName }[] = [ { id: 'home', label: 'Home', icon: 'home' }, { id: 'feed', label: 'Feed', icon: 'list' }, - { id: 'profiles', label: 'Profiles', icon: 'users' }, { id: 'compose', label: 'Compose', icon: 'edit' }, { id: 'relays', label: 'Relays', icon: 'relay' }, { id: 'signer', label: 'Signer', icon: 'key' }, diff --git a/frontend/src/test/App.test.tsx b/frontend/src/test/App.test.tsx index aeb665b..69beb1e 100644 --- a/frontend/src/test/App.test.tsx +++ b/frontend/src/test/App.test.tsx @@ -40,7 +40,7 @@ describe('App', () => { render(); await screen.findByRole('heading', { name: 'Home' }); - await user.click(screen.getByRole('button', { name: 'Profiles' })); + await user.click(screen.getByRole('button', { name: 'Manage profiles' })); await screen.findByRole('heading', { name: 'Profiles' }); const body = document.body.textContent ?? '';