From 9c84c422d07c068fa91d5a605427760a52bbbec8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nuno=20G=C3=B3is?= <github@nunogois.com>
Date: Tue, 23 Sep 2025 11:37:15 +0100
Subject: [PATCH] chore: set min height in release template dialog

---
 .../ReleasePlan/ReleasePlanReviewDialog.tsx   | 60 +++++++++++--------
 1 file changed, 36 insertions(+), 24 deletions(-)

diff --git a/frontend/src/component/feature/FeatureView/FeatureOverview/ReleasePlan/ReleasePlanReviewDialog.tsx b/frontend/src/component/feature/FeatureView/FeatureOverview/ReleasePlan/ReleasePlanReviewDialog.tsx
index 5f738842e2..7ad9abc340 100644
--- a/frontend/src/component/feature/FeatureView/FeatureOverview/ReleasePlan/ReleasePlanReviewDialog.tsx
+++ b/frontend/src/component/feature/FeatureView/FeatureOverview/ReleasePlan/ReleasePlanReviewDialog.tsx
@@ -19,10 +19,17 @@ import CloseIcon from '@mui/icons-material/Close';
 const StyledDialog = styled(Dialog)(({ theme }) => ({
     '& .MuiDialog-paper': {
         borderRadius: theme.shape.borderRadiusLarge,
-        width: theme.breakpoints.values.md,
     },
 }));
 
+const StyledScrollableContent = styled(Box)(({ theme }) => ({
+    width: theme.breakpoints.values.md,
+    minHeight: '318px',
+    overflowY: 'auto',
+    display: 'flex',
+    flexDirection: 'column',
+}));
+
 const StyledHeader = styled(Box)(({ theme }) => ({
     display: 'flex',
     justifyContent: 'space-between',
@@ -104,30 +111,35 @@ export const ReleasePlanReviewDialog = ({
                     Go back
                 </Button>
             </StyledSubHeader>
-            {activeReleasePlan && (
-                <Box sx={{ px: 4, pb: 2 }}>
-                    <Alert severity='error'>
-                        This feature environment currently has{' '}
-                        <strong>{activeReleasePlan.name}</strong> -{' '}
-                        <strong>{activeReleasePlan.milestones[0].name}</strong>
-                        {environmentEnabled ? ' running' : ' paused'}. Adding a
-                        new release plan will replace the existing release plan.
-                    </Alert>
+            <StyledScrollableContent>
+                {activeReleasePlan && (
+                    <Box sx={{ px: 4, pb: 2 }}>
+                        <Alert severity='error'>
+                            This feature environment currently has{' '}
+                            <strong>{activeReleasePlan.name}</strong> -{' '}
+                            <strong>
+                                {activeReleasePlan.milestones[0].name}
+                            </strong>
+                            {environmentEnabled ? ' running' : ' paused'}.
+                            Adding a new release plan will replace the existing
+                            release plan.
+                        </Alert>
+                    </Box>
+                )}
+                <Box sx={{ px: 2 }}>
+                    <ReleasePlan plan={planPreview} readonly />
                 </Box>
-            )}
-            <Box sx={{ px: 2 }}>
-                <ReleasePlan plan={planPreview} readonly />
-            </Box>
-            {crProtected && (
-                <Box sx={{ px: 4, pt: 1 }}>
-                    <Typography>
-                        <strong>Adding</strong> release template{' '}
-                        <strong>{template?.name}</strong> to{' '}
-                        <strong>{featureName}</strong> in{' '}
-                        <strong>{environment}</strong>.
-                    </Typography>
-                </Box>
-            )}
+                {crProtected && (
+                    <Box sx={{ px: 4, pt: 1 }}>
+                        <Typography>
+                            <strong>Adding</strong> release template{' '}
+                            <strong>{template?.name}</strong> to{' '}
+                            <strong>{featureName}</strong> in{' '}
+                            <strong>{environment}</strong>.
+                        </Typography>
+                    </Box>
+                )}
+            </StyledScrollableContent>
             <StyledDialogActions>
                 <Button variant='contained' color='primary' onClick={onConfirm}>
                     {crProtected ? 'Add suggestion to draft' : 'Apply template'}