CHECKPOINT-encryption.md

@@ -6,13 +6,12 @@ verified green at the moment this file was written.
 ## Where things are
 
 - Project: `/home/avi/Projects/Nostr_Keynctr`
-- Git repo: `master` @ `8130c3e` ("frontend: click a profile row on Home to
-   select it"). Before it: `98f4520` (active-profile picture on Compose +
-   sidebar), `f534165` (remove Profiles sidebar tab), `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` @ `98f4520` ("frontend: show active-profile picture on
+   Compose and sidebar"). Before it: `f534165` (remove Profiles sidebar tab),
+   `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.
@@ -44,7 +43,6 @@ verified green at the moment this file was written.
 
 ## Commits added most recently
 
-- `8130c3e` frontend: click a profile row on Home to select it
 - `98f4520` frontend: show active-profile picture on Compose and sidebar
 - `f534165` frontend: remove redundant Profiles sidebar tab
 - `835b0a0` frontend: list all profiles on Home, pin dev host, widen profiles card
@@ -1403,11 +1401,6 @@ A stopping point you can return to if this session is closed.
    Added `picture={active.picture}` to both the write and preview avatars in
    `ComposeScreen.tsx` and to the active-profile avatar in `Sidebar.tsx`. The
    picture now appears consistently wherever the active profile is shown.
-8. **Click a profile row on Home to select it.** Mirrored the Profiles screen's
-   clickable-row pattern: the whole non-active row in the Home "Your profiles"
-   list now selects the profile on click (guarded so Copy/Select buttons don't
-   double-fire). Added `cursor: pointer` for non-active rows. Added a test
-   asserting row-click selection.
 
 ### Files changed
 

frontend/src/screens/HomeScreen.tsx

@@ -96,16 +96,6 @@ export function HomeScreen({ onNavigate, onCreateProfile }: HomeScreenProps) {
                   <li
                     key={profile.npub}
                     className={`home-profile-row${profile.is_active ? ' is-active' : ''}`}
-                    onClick={
-                      profile.is_active
-                        ? undefined
-                        : (event) => {
-                            if ((event.target as HTMLElement).closest('button, a, input')) {
-                              return;
-                            }
-                            void onSelect(profile.npub);
-                          }
-                    }
                   >
                     <Avatar
                       npub={profile.npub}

frontend/src/styles.css

@@ -1084,10 +1084,6 @@ select {
   border-color: var(--token-border, rgba(0, 0, 0, 0.12));
 }
 
-.home-profile-row:not(.is-active) {
-  cursor: pointer;
-}
-
 .home-profile-row .active-profile-meta {
   flex: 1;
   min-width: 0;

frontend/src/test/HomeScreen.test.tsx

@@ -54,21 +54,4 @@ describe('HomeScreen', () => {
       .click(screen.getByRole('button', { name: /Create your first profile/i }));
     expect(onCreateProfile).toHaveBeenCalled();
   });
-
-  it('selects a profile when its row is clicked (not just the Select button)', async () => {
-    const backend = createFakeBackend();
-    renderHome(backend);
-    renderWithApp(<HomeScreen onNavigate={vi.fn()} onCreateProfile={vi.fn()} />);
-
-    const bobRow = (await screen.findByText('Bob')).closest('.home-profile-row') as HTMLElement;
-    // Clicking the name (not the Select button) should select the profile.
-    await userEvent.setup().click(within(bobRow).getByText('Bob'));
-
-    await waitFor(() => {
-      expect(within(bobRow).getByRole('button', { name: 'Selected' })).toBeInTheDocument();
-    });
-    // The previously active profile now offers a Select button again.
-    const aliceRow = screen.getByText('Alice').closest('.home-profile-row') as HTMLElement;
-    expect(within(aliceRow).getByRole('button', { name: 'Select' })).toBeInTheDocument();
-  });
 });