Skip to content

fix(web): Events expand shows detail under the row, not at page bottom - #37

Merged
MichaelLeeHobbs merged 1 commit into
mainfrom
fix/events-row-expand
Jul 14, 2026
Merged

fix(web): Events expand shows detail under the row, not at page bottom#37
MichaelLeeHobbs merged 1 commit into
mainfrom
fix/events-row-expand

Conversation

@MichaelLeeHobbs

Copy link
Copy Markdown
Owner

The Events page rendered expanded detail rows in a separate second .map() after all main rows, so every expanded panel appeared at the bottom of the table instead of beneath the clicked row. Fix: interleave each detail row immediately after its own main row within a single Fragment-keyed map. Web typecheck clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01F1ad6jb5mCYxzFVXHbksqi

The expanded event detail was rendered in a second .map() after all main
rows, so every expanded panel landed at the bottom of the table. Interleave
each detail row immediately after its main row inside one Fragment-keyed map.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01F1ad6jb5mCYxzFVXHbksqi
Copilot AI review requested due to automatic review settings July 14, 2026 21:39
@MichaelLeeHobbs
MichaelLeeHobbs merged commit f34d64e into main Jul 14, 2026
0 of 2 checks passed
@MichaelLeeHobbs
MichaelLeeHobbs deleted the fix/events-row-expand branch July 14, 2026 21:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Fixes the Events table row-expansion layout so the expanded detail panel renders directly beneath the clicked row (instead of being rendered in a second pass at the bottom of the table).

Changes:

  • Interleaves each event’s detail <TableRow> immediately after its corresponding main <TableRow> by mapping to a keyed Fragment.
  • Removes the separate “Expanded Detail Row” .map() that previously appended all detail rows after the main rows.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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