Clamp popover menu maxHeight to not go below POPOVER_MENU_MAX_HEIGHT

shawnborton · claude · shawnborton · commit fc006fcbebff · 2026-04-27T15:51:52.000+02:00
Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/src/components/PopoverMenu.tsx b/src/components/PopoverMenu.tsx
@@ -556,7 +556,7 @@ function BasePopoverMenu({
         const stylesArray: ViewStyle[] = [StyleSheet.flatten(styles.createMenuContainer), {width: variables.compactPopoverMenuWidth}, styles.pv2];
 
         if (shouldUseScrollView && shouldEnableMaxHeight && !isInLandscapeMode) {
-            stylesArray.push({maxHeight: windowHeight - variables.compactPopoverMenuVerticalMargin});
+            stylesArray.push({maxHeight: Math.max(windowHeight - variables.compactPopoverMenuVerticalMargin, CONST.POPOVER_MENU_MAX_HEIGHT)});
         }
 
         return stylesArray;

Original file line number	Diff line number	Diff line change
`@@ -556,7 +556,7 @@ function BasePopoverMenu({`
`556`	`556`	`const stylesArray: ViewStyle[] = [StyleSheet.flatten(styles.createMenuContainer), {width: variables.compactPopoverMenuWidth}, styles.pv2];`
`557`	`557`
`558`	`558`	`if (shouldUseScrollView && shouldEnableMaxHeight && !isInLandscapeMode) {`
`559`		`- stylesArray.push({maxHeight: windowHeight - variables.compactPopoverMenuVerticalMargin});`
	`559`	`+ stylesArray.push({maxHeight: Math.max(windowHeight - variables.compactPopoverMenuVerticalMargin, CONST.POPOVER_MENU_MAX_HEIGHT)});`
`560`	`560`	`}`
`561`	`561`
`562`	`562`	`return stylesArray;`