Skip to content

feat(app): add controls to sessions on home page - #46655

Open
kostaris wants to merge 4 commits into
anomalyco:devfrom
kostaris:home-session-menu
Open

feat(app): add controls to sessions on home page#46655
kostaris wants to merge 4 commits into
anomalyco:devfrom
kostaris:home-session-menu

Conversation

@kostaris

@kostaris kostaris commented Sep 1, 2026

Copy link
Copy Markdown

Issue for this PR

Fixes web-ui home page not having controls for each session

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

  1. Extracts reusable SessionActionMenu, SessionDeleteDialog, and SessionSharePopover components.
  2. Adds session action menu (rename, share/unshare, export, archive, delete) to home session list rows.
  3. Unifies session timeline header actions with the extracted shared components.
  4. Makes share popover state reactive to sync store updates.
  5. Prevents delete confirmation dialog from closing on deletion failure.
  6. Adds unit tests for session actions and descendant deletion.

How did you verify your code works?

  • Ran bun test packages/app/src/pages/home/home-session-menu.test.ts.
  • Ran bun turbo run typecheck.
  • Tested rename, share popover, export, archive, and delete from home screen.

Screenshots / recordings

2026-09-01_21-50

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@kostaris kostaris changed the title feat(app): add home session menu feat(app): add controls to sessions on home page Sep 1, 2026
@kostaris
kostaris force-pushed the home-session-menu branch 2 times, most recently from be80d42 to d6068ce Compare September 2, 2026 22:27
@carthageadev

carthageadev commented Sep 2, 2026

Copy link
Copy Markdown

Yo twin, could we also support right-click -> Rename on Home session rows? Renaming tabs already works that way, so it'd fit the existing design. Your menu button and right-click could both stay. I've got a small patch with tests in #46915 and would be happy to coordinate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants