Skip to content

HOLD: Sync event dashboard breakdown cards across a row - #2052

Draft
maebeale wants to merge 1 commit into
mainfrom
maebeale/sync-row-data-toggles
Draft

HOLD: Sync event dashboard breakdown cards across a row#2052
maebeale wants to merge 1 commit into
mainfrom
maebeale/sync-row-data-toggles

Conversation

@maebeale

@maebeale maebeale commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator

🤖 suggested review level: 3 Read 📖 small progressive-enhancement Stimulus controller + two dashboard rows wired to it

What

Toggling one breakdown card on the event dashboard now opens/closes the rest of the cards in that row.

Why

  • Cards are native <details> grouped into rows (money row + headcount row); expanding one left the others collapsed, forcing a click per card to compare categories.
  • New details_row_sync Stimulus controller mirrors a card's open state onto its row siblings. Applied independently per row, so the money row and headcount row toggle separately. Nested money-row <details> are unaffected (native toggle doesn't bubble).

Test

  • New JS system spec: clicking one headcount card opens all cards in the row, clicking again closes them. Verified it fails without the controller wired.

The event dashboard groups its breakdown cards into rows of native
<details>. Expanding one card left the others closed, so seeing every
category's detail meant clicking each card individually. Mirror the
open/closed state across the row instead: toggling one card toggles the
rest, applied independently to the money row and the headcount row.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings August 2, 2026 18:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@maebeale maebeale changed the title Sync event dashboard breakdown cards across a row HOLD: Sync event dashboard breakdown cards across a row Aug 2, 2026
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