Fix/blabsy create message add scroll

[sosweetham]

Description of change

adds scroll bar to create message modal when making groups with a lot of members

Issue Number

n/a

Type of change

• Fix (a change which fixes an issue)

How the change has been tested

n/a

Change checklist

• I have ensured that the CI Checks pass locally
• I have removed any unnecessary logic
• My code is well documented
• I have signed my commits
• My code follows the pattern of the application
• I have self reviewed my code

Summary by CodeRabbit

• Style
  • Improved scrolling behavior in the Add Members modal for better navigation when viewing extended member lists.

[coderabbitai]

📝 Walkthrough

Walkthrough

The AddMembers modal layout is restructured to improve scrolling behavior. Dialog.Panel now uses flexbox with vertical scroll support, a new wrapper div enables content scrolling, and spacing classes are adjusted for consistency. No business logic or API changes.

Changes

Cohort / File(s)	Summary
Modal Layout & Scrolling platforms/blabsy/src/components/chat/add-members.tsx	Restructured Dialog.Panel with flexbox column layout and max-height constraint. Added nested wrapper div with overflow-y-auto for improved vertical scrolling. Adjusted Dialog.Title, search/results container, and selected-members section sizing to support better scroll behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Suggested reviewers

• coodos

Poem

🐰 Hop along this scrolly path,
A modal flows with layout's wrath,
Flexbox bends, divs arrange,
Scrolling smooth—no logic change!
Pretty wraps, the UI grows,
Much to see as content flows! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Merge Conflict Detection	⚠️ Warning	❌ Merge conflicts detected (7 files): ⚔️ infrastructure/dev-sandbox/README.md (content) ⚔️ infrastructure/dev-sandbox/package.json (content) ⚔️ infrastructure/dev-sandbox/src/routes/+page.svelte (content) ⚔️ infrastructure/eid-wallet/src/routes/(app)/main/+page.svelte (content) ⚔️ platforms/blabsy/src/components/chat/add-members.tsx (content) ⚔️ platforms/registry/motd.json (content) ⚔️ pnpm-lock.yaml (content) These conflicts must be resolved before merging into main.	Resolve conflicts locally and push changes to this branch.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Fix/blabsy create message add scroll' directly addresses the main change: adding scroll functionality to the create message modal.
Description check	✅ Passed	The PR description covers the required sections: change description, issue number, type of change, testing approach, and a completed checklist. All essential information is present.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/blabsy-create-message-add-scroll

⚔️ Resolve merge conflicts (beta)

• Auto-commit resolved conflicts to branch fix/blabsy-create-message-add-scroll
• Create stacked PR with resolved conflicts
• Post resolved changes as copyable diffs in a comment

---

No actionable comments were generated in the recent review. 🎉

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}