Skip to content

Conversation

@mirhamasala
Copy link
Collaborator

@mirhamasala mirhamasala commented Jan 5, 2026

📝 Description

This PR wraps the ServiceProvidersTable component in a BreakoutContainer from the UI library to improve the table's layout and allow it to break out of its parent container constraints.

UXIT-3810

Fixes #191

🛠️ Key Changes

  • Imported BreakoutContainer from @filecoin-foundation/ui-filecoin/Container
  • Wrapped the table component with BreakoutContainer in the search results section

📌 To-Do Before Merging

🧪 How to Test

  • Setup: Run the development server locally
  • Steps to Test:
    1. Navigate to the service providers page
    2. Verify the table displays correctly
    3. Test search functionality and confirm the table still renders properly with filtered results
    4. Check responsive behavior on different screen sizes
  • Expected Results: The table should display within a BreakoutContainer, potentially with improved spacing and layout
  • Additional Notes: The changes are currently commented out, so they will need to be uncommented before merging

📸 Screenshots

CleanShot 2026-01-05 at 15 39 31@2x

🔖 Resources

@vercel
Copy link

vercel bot commented Jan 5, 2026

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

Project Deployment Review Updated (UTC)
filecoin-cloud Ready Ready Preview, Comment Jan 5, 2026 3:41pm

@mirhamasala mirhamasala marked this pull request as ready for review January 5, 2026 15:40
@mirhamasala mirhamasala force-pushed the mm/use-breakout-container branch from a06c41b to b01335a Compare January 5, 2026 15:40
@github-actions github-actions bot added size/M and removed size/S labels Jan 5, 2026
@mirhamasala mirhamasala merged commit a3e0f56 into main Jan 5, 2026
6 checks passed
@mirhamasala mirhamasala deleted the mm/use-breakout-container branch January 5, 2026 15:43
@github-project-automation github-project-automation bot moved this from 📌 Triage to 🎉 Done in FS Jan 5, 2026
@github-actions github-actions bot added size/M and removed size/M labels Jan 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

Fix tables width to use full window width

2 participants