avi/Keynctr
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

frontend: remove redundant Profiles sidebar tab

Browse source 
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.
This commit is contained in:
Avi 2026-08-26 14:49:52 -05:00
commit f53416528c
2 changed files with 1 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

1
frontend/src/components/Sidebar.tsx
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' },

2
frontend/src/test/App.test.tsx
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 ?? '';