fix(ui): stop event propagation on observation tab buttons #126

Merged

jonas.hanisch merged 1 commit from fix/123-tab-event-propagation into main

2026-01-24 11:58:45 +00:00

jack commented

2026-01-24 09:33:21 +00:00

Owner

Summary

Fixes tab buttons collapsing the observation card when clicked
Added e.stopPropagation() to both Narrative and Details tab buttons

Problem

In the Memories view, when clicking on a tab (Narrative/Details) to switch views, the click event bubbled up to the card's onClick handler and collapsed the expanded card.

Solution

Added e.stopPropagation() to the tab button onClick handlers to prevent the event from bubbling up.

Test plan

Open Memories view
Expand an observation card
Click on "Details" tab - card should stay expanded
Click on "Narrative" tab - card should stay expanded
Click outside the tabs on the card header - card should collapse (unchanged behavior)

Closes #123

🤖 Generated with Claude Code

## Summary - Fixes tab buttons collapsing the observation card when clicked - Added `e.stopPropagation()` to both Narrative and Details tab buttons ## Problem In the Memories view, when clicking on a tab (Narrative/Details) to switch views, the click event bubbled up to the card's onClick handler and collapsed the expanded card. ## Solution Added `e.stopPropagation()` to the tab button onClick handlers to prevent the event from bubbling up. ## Test plan - [ ] Open Memories view - [ ] Expand an observation card - [ ] Click on "Details" tab - card should stay expanded - [ ] Click on "Narrative" tab - card should stay expanded - [ ] Click outside the tabs on the card header - card should collapse (unchanged behavior) Closes #123 🤖 Generated with [Claude Code](https://claude.com/claude-code)

jack added 2 commits

2026-01-24 09:33:21 +00:00

fix: MCP server respects remote backend configuration

CI / check (pull_request) Successful in 0s

Details

CI / build (pull_request) Successful in 41s

Details

CI / validate-plugin (pull_request) Successful in 36s

Details

280ca611f3

The MCP server was ignoring REMOTE_MODE, REMOTE_URL, and REMOTE_TOKEN
settings, always connecting to localhost. This fix:

- Loads settings via loadSettings() from @claude-mem/shared
- Uses REMOTE_URL when REMOTE_MODE is enabled
- Adds Authorization header with REMOTE_TOKEN for all API calls
- Falls back to localhost with env var overrides in local mode

Fixes #122

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

fix(ui): stop event propagation on observation tab buttons

CI / check (pull_request) Successful in 1s

Details

CI / build (pull_request) Successful in 42s

Details

CI / validate-plugin (pull_request) Successful in 35s

Details

d1d58ff2dc

Clicking the Narrative/Details tabs in an expanded observation card
caused the card to collapse because the click event bubbled up to
the card's onClick handler.

Added e.stopPropagation() to both tab buttons to prevent this.

Fixes #123

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>