From 3211c62e6d4c599a6a1352f4e20073ba4762f211 Mon Sep 17 00:00:00 2001 From: Vineet Sharma Date: Wed, 22 Jul 2026 09:51:21 +0530 Subject: [PATCH] fix(oc-docs): cap menu dropdown height so long lists scroll The assertion operator dropdown lists 28 options; the shared MenuDropdown surface was capped at 90vh, which measures the browser viewport rather than the embedded component, so the list spilled past the app. Cap the surface at min(20rem, 90vh) so long lists scroll instead of overflowing. --- packages/oc-docs/src/ui/MenuDropdown/StyledWrapper.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/packages/oc-docs/src/ui/MenuDropdown/StyledWrapper.ts b/packages/oc-docs/src/ui/MenuDropdown/StyledWrapper.ts index 471095cc..91bf9490 100644 --- a/packages/oc-docs/src/ui/MenuDropdown/StyledWrapper.ts +++ b/packages/oc-docs/src/ui/MenuDropdown/StyledWrapper.ts @@ -16,7 +16,10 @@ export const StyledWrapper = styled.div` box-shadow: var(--oc-dropdown-shadow); border-radius: var(--oc-border-radius-base); border: 1px solid var(--oc-dropdown-border); - max-height: 90vh; + /* Cap the surface height so a long list (e.g. the assertion operators) scrolls + instead of spilling past the embedded component; min() keeps it within small + viewports too. */ + max-height: min(20rem, 90vh); overflow-y: auto; max-width: unset !important; padding: 0.25rem;