Skip to content

Fix space selector overflow#955

Merged
Dhravya merged 2 commits into
mainfrom
Dhravya/fix-space-text-cutoff
May 17, 2026
Merged

Fix space selector overflow#955
Dhravya merged 2 commits into
mainfrom
Dhravya/fix-space-text-cutoff

Conversation

@Dhravya
Copy link
Copy Markdown
Member

@Dhravya Dhravya commented May 17, 2026

Summary

  • Constrain the space selector modal so it cannot grow past the viewport.
  • Prevent horizontal overflow in the sidebar and spaces list panes so long names or counts do not get clipped.
image

Testing

  • bunx biome check apps/web/components/select-spaces-modal.tsx

@claude
Copy link
Copy Markdown

claude Bot commented May 17, 2026

Claude finished @Dhravya's task in 23s —— View job


Review Complete

  • Analyzed PR changes
  • Reviewed CSS overflow fixes

Overview: Adds CSS constraints to prevent the space selector modal and its child containers from overflowing the viewport.

Issues found: None — this looks good to ship.

The changes apply standard flexbox overflow fixes:

  • min-w-0 on flex containers allows them to shrink below their content size (required for overflow to work on flex children)
  • overflow-x-hidden on the sidebar and spaces list panes prevents horizontal scroll/clipping
  • overflow-hidden on the main content flex container ensures proper containment

These are well-established CSS patterns for handling overflow in flexbox layouts.

Score: 10/10


✅ This PR has been merged.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 17, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
supermemory-mcp c2c2e1d May 17 2026, 07:03 AM

Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Overview: Adds CSS constraints to prevent the space selector modal and its child containers from overflowing the viewport.

Issues found: None — this looks good to ship.

The changes apply standard flexbox overflow fixes:

  • min-w-0 on flex containers allows them to shrink below their content size (required for overflow to work on flex children)
  • overflow-x-hidden on the sidebar and spaces list panes prevents horizontal scroll/clipping
  • overflow-hidden on the main content flex container ensures proper containment

These are well-established CSS patterns for handling overflow in flexbox layouts.

Score: 10/10

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 17, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
supermemory-app c2c2e1d Commit Preview URL

Branch Preview URL
May 17 2026, 07:04 AM

@Dhravya Dhravya merged commit 6df9c8f into main May 17, 2026
5 of 7 checks passed
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.

1 participant