Skip to content

Comments

fix: updated the prompt dialog height relative to the screen size#603

Merged
Fermionic-Lyu merged 1 commit intoInsForge:mainfrom
Abiji-2020:fix/ui
Nov 21, 2025
Merged

fix: updated the prompt dialog height relative to the screen size#603
Fermionic-Lyu merged 1 commit intoInsForge:mainfrom
Abiji-2020:fix/ui

Conversation

@Abiji-2020
Copy link
Contributor

@Abiji-2020 Abiji-2020 commented Nov 19, 2025

Summary

Modified the Prompt dialog component to have a relative screen size and made it scroll able for large sizes.
fixes #602

How did you test this change?

scroll able dialog

image image

Summary by CodeRabbit

  • Bug Fixes
    • Fixed dialog content overflow: dialogs now properly display content that exceeds the visible area, with scrolling enabled.

…e screen size

Signed-off-by: Abinand P <abinand0911@gmail.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 19, 2025

Walkthrough

Modified DialogContent styling in PromptDialog component to enable vertical scrolling within a constrained viewport height. Changed overflow behavior from hidden to auto and added a maximum height constraint of 90vh to ensure content remains accessible.

Changes

Cohort / File(s) Change Summary
Dialog scrolling fix
frontend/src/features/dashboard/components/PromptDialog.tsx
Updated DialogContent styling: replaced overflow-hidden with overflow-y-auto and added max-h-[90vh] constraint to enable scrollable content display within viewport bounds

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Single file with focused CSS styling changes
  • No logic modifications or complex interactions
  • Straightforward overflow and height property adjustments

Poem

🐰 A scrollbar appears, oh what delight!
Content no longer hidden from sight,
At ninety percent, the height is just right,
Prompts now visible—bugs take their flight! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed The PR successfully addresses issue #602 by making the PromptDialog scrollable with max-h-[90vh] and enabling vertical scrolling, ensuring UI elements are visible in cloud environments.
Out of Scope Changes check ✅ Passed All changes are directly scoped to fixing the PromptDialog visibility issue in #602; only styling adjustments to DialogContent were made with no extraneous modifications.
Title check ✅ Passed The title accurately and concisely describes the main change—adjusting dialog height to be relative to screen size, which directly addresses the core issue of visibility in the cloud environment.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 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

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

@Abiji-2020 Abiji-2020 changed the title feat: updated the prompt dialog height relative to the screen size fix: updated the prompt dialog height relative to the screen size Nov 19, 2025
@Fermionic-Lyu
Copy link
Contributor

@Abiji-2020 Hi Abiji, thanks for reporting this issue. May I ask what's your screen resolution and which browser did you use? This helps our responsive design. Thanks!

@Abiji-2020
Copy link
Contributor Author

@Abiji-2020 Hi Abiji, thanks for reporting this issue. May I ask what's your screen resolution and which browser did you use? This helps our responsive design. Thanks!

I am using firefox browser and the resolution is 1366x768 (16:9) ratio

Copy link
Contributor

@Fermionic-Lyu Fermionic-Lyu left a comment

Choose a reason for hiding this comment

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

This is a quick solution to the responsive issue in smaller screens. We will revisit the responsive design later.

@Fermionic-Lyu Fermionic-Lyu merged commit dce0432 into InsForge:main Nov 21, 2025
4 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.

[Bug]: Quick start prompt is not fully visible in the cloud environment

2 participants