fix: updated the prompt dialog height relative to the screen size#603
fix: updated the prompt dialog height relative to the screen size#603Fermionic-Lyu merged 1 commit intoInsForge:mainfrom
Conversation
…e screen size Signed-off-by: Abinand P <abinand0911@gmail.com>
WalkthroughModified 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
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
|
@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 |
Fermionic-Lyu
left a comment
There was a problem hiding this comment.
This is a quick solution to the responsive issue in smaller screens. We will revisit the responsive design later.
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
Summary by CodeRabbit