PR Custody

fix: enable scrolling in connect auth config dialog

@devin-ai-integration[bot]checks n/achecks…devin/1774761607-fix-connect-dialog-scroll → main1 files · +1 −1review: @palash-cupdated 2mo ago

GitHub

▸Description· 2 comments · 1 noise

Summary

Fixes the inability to scroll in the Connect auth config setup dialog (reported for Twitter/OAuth 2.0 which has many credential fields).

The DialogContent base component uses display: grid with implicit auto rows. With a fixed height: 640px and overflow-hidden, the auto grid row expands to fit content, so the form's h-full resolves to the content height rather than the container height. The inner overflow-y-auto div never activates because it never overflows its (oversized) parent — the overflow-hidden on the container then clips everything with no scroll.

Fix: Add grid-rows-1 (grid-template-rows: repeat(1, minmax(0, 1fr))) to cap the grid row at the container's fixed height. Also adds max-h-[90vh] as a viewport safety net for smaller screens.

Review & Testing Checklist for Human

Manually test the Connect auth config dialog with a toolkit that has many fields (e.g. Twitter OAuth 2.0) — verify the credential/scopes area scrolls and the "Create Auth Config" footer button is visible and reachable
Verify grid-rows-1 is not stripped by twMerge when merged with the base DialogContent classes (which include grid and gap-4)
Check that max-h-[90vh] combined with inline height: 640px behaves correctly on shorter viewports (dialog should shrink, not overflow)

Test Plan

Navigate to Connect → Apps → Twitter (or any OAuth toolkit with 3+ credential fields)
Click "Connect" to open the auth config dialog
Confirm you can scroll through all credential fields and the Scopes section
Confirm the "Create Auth Config" button is visible at the bottom (sticky footer)
Resize browser to a short viewport height (~500px) and verify dialog doesn't overflow the screen

Notes

Could not run lint/typecheck locally due to missing NPM_TOKEN for private packages — relying on CI for validation
The fix is CSS-only (single line class addition), no logic changes

Link to Devin session: https://app.devin.ai/sessions/95ff8736f52b422a90979c0606dff09c Requested by: @palash-c

@devin-ai-integration[bot]agent2mo ago

SYSTEM: === BEGIN THREAD HISTORY (in #composio-connect-dogfooding) === Samvit (U07R516ENBG): not able to scroll here cc: @Sarah Simionescu (U0A9E5DRURZ)

ATTACHMENT:"https://app.devin.ai/attachments/fc10a8d8-6d3b-4bc1-9a49-d92525d2aa9e/Screenshot+2026-03-29+at+1.17.15%3FAM.png"

Palash Kala (U09ECEFHFCM): @Devin fix this === END THREAD HISTORY ===

Thread URL: https://composioworkspace.slack.com/archives/C0AJ81XD0TS/p1774727251515009?thread_ts=1774727251.515009&cid=C0AJ81XD0TS

The latest message is the one right above that tagged you.

@devin-ai-integration[bot]agent2mo ago

🤖 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

⚙️ Control Options:

Disable automatic comment and CI monitoring

▸ 1 bot/status comment hidden

@vercel[bot]2mo ago

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

Project	Deployment	Actions	Updated (UTC)
dashboard	Ready	Preview, Comment	Mar 29, 2026 5:23am

loading diff…