Skip to content

fix: make events empty state full width in grid table#339

Merged
ben-fornefeld merged 4 commits into
mainfrom
devin/1779425280-fix-events-empty-state-width
May 22, 2026
Merged

fix: make events empty state full width in grid table#339
ben-fornefeld merged 4 commits into
mainfrom
devin/1779425280-fix-events-empty-state-width

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented May 22, 2026

Copy link
Copy Markdown
Contributor

Summary

The "No events found" empty state in the sandbox events table was not stretching to full width. The events table uses display: grid for virtualization (same as sandbox logs and build logs), but the shared TableEmptyState component is designed for standard table-fixed layouts where colSpan works natively.

Fix: Extracted a new VirtualizedTableEmptyState component in virtualized-table-ui.tsx that uses the same flex min-w-full pattern as VirtualizedTableLoaderBody and VirtualizedTableRow. The core TableEmptyState is left untouched so non-virtualized tables (api-keys, invoices, members, webhooks) continue working correctly.

Review & Testing Checklist for Human

  • Navigate to a sandbox detail page → Events tab with no events and verify the empty state rows span the full width of the table
  • Verify the loading state still looks correct (uses VirtualizedTableLoaderBody — unchanged)
  • Spot-check that other empty states (API keys, invoices, members) still render correctly

Notes

  • All virtualized tables (logs, build logs, events) use display: grid on <Table> — this is required for absolute-positioned virtualized rows. Non-virtualized tables use standard table-fixed layout.
  • The logs/build logs empty state (LogsEmptyBody) uses a different visual style (tall centered text) and was not changed.

Link to Devin session: https://app.devin.ai/sessions/286135bb1f0e49c0a49cd1512d67be3a
Requested by: @mlejva

Co-Authored-By: vasek <vasek.mlejnsky@gmail.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@cla-bot cla-bot Bot added the cla-signed label May 22, 2026
@vercel

vercel Bot commented May 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
web Ready Ready Preview, Comment May 22, 2026 9:28pm
web-juliett Ready Ready Preview, Comment May 22, 2026 9:28pm

Request Review

Co-Authored-By: vasek <vasek.mlejnsky@gmail.com>
Create a dedicated VirtualizedTableEmptyState component that uses the
flex min-w-full pattern required by grid-based virtualized tables.
This keeps the core TableEmptyState untouched for standard table layouts.

Co-Authored-By: vasek <vasek.mlejnsky@gmail.com>
@ben-fornefeld ben-fornefeld merged commit c40452d into main May 22, 2026
14 checks passed
@ben-fornefeld ben-fornefeld deleted the devin/1779425280-fix-events-empty-state-width branch May 22, 2026 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants