💄(settings modal) updated the settings modal size

[elvoisin]

fix modal settings size

Summary by CodeRabbit

• Bug Fixes
  • Fixed settings modal sizing to properly adapt to different viewport heights.

[coderabbitai]

Warning

Rate limit exceeded

@elvoisin has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 36 minutes and 53 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 9d243283-b33d-4cb2-8909-d48d74352551

📥 Commits

Reviewing files that changed from the base of the PR and between 5a924f7 and cdba45b.

📒 Files selected for processing (2)

• CHANGELOG.md
• src/frontend/apps/conversations/src/features/settings/components/SettingsModal.tsx

Walkthrough

This PR updates the SettingsModal component to enforce a minimum height using viewport-relative constraints, refactoring the Modal prop structure from object spreading to inline JSX. The changelog is updated to document this fix in a new [Unreleased] section.

Changes

Settings Modal Height Constraint

Layer / File(s)	Summary
Settings modal height constraint implementation src/frontend/apps/conversations/src/features/settings/components/SettingsModal.tsx	Modal rendering is refactored to pass props inline instead of spreading a tabModalProps object, and constraints.minHeight is added to enforce viewport-relative minimum height.
Changelog entry for settings modal fix CHANGELOG.md	A new [Unreleased] section with a Fixed subsection documents the settings modal size update.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• suitenumerique/conversations#457: Adjusts SettingsModal.tsx Modal rendering props and adds viewport-based constraints for minimum height, extending prior settings modal work.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically addresses the main change: updating the settings modal size, which matches the primary modification in SettingsModal.tsx.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch evoisin/fix-modal-settings-size

---

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.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud