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

Merged
jack merged 6 commits from fix/126-stop-propagation-tabs into main 2026-01-24 13:06:24 +00:00
Owner

Summary

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

Problem

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.

Closes #126

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
## Summary - Fixes tab buttons collapsing the observation card when clicked - Added `e.stopPropagation()` to both Narrative and Details tab buttons ## Problem 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. Closes #126 ## 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
fix(ui): stop event propagation on observation tab buttons
All checks were successful
CI / check (pull_request) Successful in 0s
CI / build (pull_request) Successful in 48s
CI / validate-plugin (pull_request) Successful in 1m3s
cfd60efdf2
Fixes tab buttons collapsing the observation card when clicked.
Added e.stopPropagation() to both Narrative and Details tab buttons.

Closes #126

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Merge branch 'main' into fix/126-stop-propagation-tabs
All checks were successful
CI / build (pull_request) Successful in 1m19s
06bd6f4b1d
Merge branch 'main' into fix/126-stop-propagation-tabs
All checks were successful
CI / build (pull_request) Successful in 1m16s
d1870bfa06
Merge branch 'main' into fix/126-stop-propagation-tabs
All checks were successful
CI / build (pull_request) Successful in 1m7s
f7a4ffaedb
Merge branch 'main' into fix/126-stop-propagation-tabs
All checks were successful
CI / build (pull_request) Successful in 1m19s
b46e88f017
Merge branch 'main' into fix/126-stop-propagation-tabs
All checks were successful
CI / build (pull_request) Successful in 1m14s
79a49a8ce8
jack merged commit 59b7d03cbe into main 2026-01-24 13:06:24 +00:00
Sign in to join this conversation.
No description provided.