Compare commits

...

2 commits

Author SHA1 Message Date
Avi
10148644e7 checkpoint: remove Profiles sidebar tab 2026-08-26 14:50:07 -05:00
Avi
f53416528c frontend: remove redundant Profiles sidebar tab
The Profiles screen is still reachable via Home's 'Manage profiles'
button, so the left-nav Profiles tab was redundant. Dropped the
'profiles' entry from the sidebar NAV_ITEMS and updated App.test.tsx
to navigate to the screen via 'Manage profiles' instead.
2026-08-26 14:49:52 -05:00
3 changed files with 14 additions and 7 deletions

View file

@ -6,11 +6,12 @@ verified green at the moment this file was written.
## Where things are ## Where things are
- Project: `/home/avi/Projects/Nostr_Keynctr` - Project: `/home/avi/Projects/Nostr_Keynctr`
- Git repo: `master` @ `835b0a0` ("frontend: list all profiles on Home, pin - Git repo: `master` @ `f534165` ("frontend: remove redundant Profiles sidebar
dev host, widen profiles card"). Before it: `71e580f` (Trim white margins tab"). Before it: `835b0a0` (list all profiles on Home, pin dev host,
from sidebar logo artwork), `f29d4f9` (Checkpoint 0.45 migration), widen card), `5635f79` (checkpoint), `71e580f` (Trim white margins),
`fa5ba08` (Security: upgrade nostr stack 0.40->0.45), `bf10ae3`, `e210f17`, `f29d4f9` (Checkpoint 0.45 migration), `fa5ba08` (Security: upgrade nostr
`7098e75`, `e6a1efc`, `6d5063d`, `1c428a9`. stack 0.40->0.45), `bf10ae3`, `e210f17`, `7098e75`, `e6a1efc`, `6d5063d`,
`1c428a9`.
- Working tree: still DIRTY with the long-standing user changes — `concept3.svg` - Working tree: still DIRTY with the long-standing user changes — `concept3.svg`
deleted, `KeynectrAppIconPossibility02.jpeg` untracked. These are NOT part deleted, `KeynectrAppIconPossibility02.jpeg` untracked. These are NOT part
of the frontend commit above and remain uncommitted. 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 ## Commits added most recently
- `f534165` frontend: remove redundant Profiles sidebar tab
- `835b0a0` frontend: list all profiles on Home, pin dev host, widen profiles card - `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 - `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 half. Changed it to a single `1fr` column and dropped the now-redundant
`@media (max-width: 900px)` override, so "Your profiles" matches the width `@media (max-width: 900px)` override, so "Your profiles" matches the width
of "Most recent publication" below it. No JSX change. 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 ### Files changed

View file

@ -9,7 +9,6 @@ import logoUrl from '../assets/logo.png';
const NAV_ITEMS: { id: Screen; label: string; icon: IconName }[] = [ const NAV_ITEMS: { id: Screen; label: string; icon: IconName }[] = [
{ id: 'home', label: 'Home', icon: 'home' }, { id: 'home', label: 'Home', icon: 'home' },
{ id: 'feed', label: 'Feed', icon: 'list' }, { id: 'feed', label: 'Feed', icon: 'list' },
{ id: 'profiles', label: 'Profiles', icon: 'users' },
{ id: 'compose', label: 'Compose', icon: 'edit' }, { id: 'compose', label: 'Compose', icon: 'edit' },
{ id: 'relays', label: 'Relays', icon: 'relay' }, { id: 'relays', label: 'Relays', icon: 'relay' },
{ id: 'signer', label: 'Signer', icon: 'key' }, { id: 'signer', label: 'Signer', icon: 'key' },

View file

@ -40,7 +40,7 @@ describe('App', () => {
render(<App />); render(<App />);
await screen.findByRole('heading', { name: 'Home' }); 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' }); await screen.findByRole('heading', { name: 'Profiles' });
const body = document.body.textContent ?? ''; const body = document.body.textContent ?? '';