diff --git a/CHECKPOINT-encryption.md b/CHECKPOINT-encryption.md index fad8a95..643445a 100644 --- a/CHECKPOINT-encryption.md +++ b/CHECKPOINT-encryption.md @@ -6,12 +6,11 @@ verified green at the moment this file was written. ## Where things are - Project: `/home/avi/Projects/Nostr_Keynctr` -- 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`. +- 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`. - 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. @@ -43,7 +42,6 @@ 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 @@ -1389,12 +1387,6 @@ 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 5a4aab7..cce4bca 100644 --- a/frontend/src/components/Sidebar.tsx +++ b/frontend/src/components/Sidebar.tsx @@ -9,6 +9,7 @@ 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 69beb1e..aeb665b 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: 'Manage profiles' })); + await user.click(screen.getByRole('button', { name: 'Profiles' })); await screen.findByRole('heading', { name: 'Profiles' }); const body = document.body.textContent ?? '';