From 2e7afba3a0fe52574177a2971fb7efcee40bb7a0 Mon Sep 17 00:00:00 2001 From: benjdlambert Date: Thu, 24 Jul 2025 13:30:33 +0200 Subject: [PATCH] chore: adjust the comment and fix Signed-off-by: benjdlambert --- plugins/catalog/src/alpha/DefaultEntityContentLayout.tsx | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/plugins/catalog/src/alpha/DefaultEntityContentLayout.tsx b/plugins/catalog/src/alpha/DefaultEntityContentLayout.tsx index a18c3fbab3..e5f7007356 100644 --- a/plugins/catalog/src/alpha/DefaultEntityContentLayout.tsx +++ b/plugins/catalog/src/alpha/DefaultEntityContentLayout.tsx @@ -101,9 +101,10 @@ const useStyles = makeStyles< gridArea: 'info', position: 'sticky', top: theme.spacing(3), - // this is a little unfortunate, it's basically vh - header, in order for the page to scroll - // naturally throught the main cards. - maxHeight: `calc(100vh - ${theme.spacing(6)}px)`, + // this is a little unfortunate, but it's required to make the info cards scrollable + // in a fixed container of the full height when it's stuck. + // 100% doesn't work as that's the height of the entire layout, which is what powers the card scrolling. + maxHeight: '100vh', overflowY: 'auto', alignSelf: 'start', // Hide the scrollbar for the inner info cards