fix: Cloud Config parameter modal cannot scroll when content scales beyond visible area

[mtrezza]

New Pull Request Checklist

• I am not disclosing a vulnerability.
• I am creating this PR in reference to an issue.

Summary by CodeRabbit

Release Notes

• Style
  • Updated modal sections to display scrollable content when overflow occurs.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[parse-github-assistant]

🚀 Thanks for opening this pull request! We appreciate your effort in improving the project. Please let us know once your pull request is ready for review.

[parseplatformorg]

✅ Snyk checks have passed. No issues have been found so far.

Status	Scanner	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

[coderabbitai]

Warning

Rate limit exceeded

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

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

📥 Commits

Reviewing files that changed from the base of the PR and between e3141da and f2b4c7e.

📒 Files selected for processing (1)

• src/components/Modal/Modal.scss

📝 Walkthrough

Walkthrough

A single styling rule is added to the Modal component to handle overflow behavior. Direct children of .modal elements (excluding first and last) now set overflow: auto to enable scrolling when content exceeds container bounds.

Changes

Cohort / File(s)	Summary
Modal component styling src/components/Modal/Modal.scss	Added overflow: auto CSS rule for inner modal sections to enable scroll behavior on overflowing content

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is incomplete. Only the checklist items are filled out, but critical sections like Issue Description, Approach, and TODOs are entirely missing.	Add Issue Description section explaining the problem, Approach section detailing the solution, and complete any applicable TODOs before merging.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the primary change: fixing a modal scrolling issue when content exceeds the visible area.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/components/Modal/Modal.scss (1)

26-31: Add min-height: 0 to enable scrolling in the flex layout.

In a flex column layout with fixed-size header and footer (both flex-shrink: 0), the middle content section needs min-height: 0 to shrink below its content size—otherwise overflow: auto won't scroll. The middle children currently default to min-height: auto, which prevents shrinking and blocks scrolling.

Required fix

  > *:not(:first-child):not(:last-child) {
    border-color: $borderGrey;
    border-width: 0 0 1px 0;
    border-style: solid;
    overflow: auto;
+   min-height: 0;
  }

[parseplatformorg]

🎉 This change has been released in version 8.3.0-alpha.7