implement new size prop with a fixed height

WordPress · Oct 26, 2023 · abbf27e · abbf27e
1 parent 820b135
commit abbf27e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/packages/edit-site/src/components/global-styles/font-library-modal/index.js b/packages/edit-site/src/components/global-styles/font-library-modal/index.js
@@ -51,7 +51,7 @@ function FontLibraryModal( {
 		<Modal
 			title={ __( 'Fonts' ) }
 			onRequestClose={ onRequestClose }
-			isFullScreen
+			size="large"
 			className="font-library-modal"
 		>
 			<TabPanel

diff --git a/packages/edit-site/src/components/global-styles/font-library-modal/style.scss b/packages/edit-site/src/components/global-styles/font-library-modal/style.scss
@@ -1,11 +1,12 @@
 .font-library-modal {
-	// @todo: If a new prop is added to the Modal component that constrains
-	// the content width, we should use that prop instead of this style.
-	// see https://github.com/WordPress/gutenberg/issues/54471.
+	// @todo: If a new prop is added to the Modal component that fixes the height
+	// and/or constrains the origin, we should use that instead of this style.
+	// see https://github.com/WordPress/gutenberg/issues/55062.
 	&.font-library-modal {
 
 		@include break-medium() {
-			width: 65vw;
+			height: calc(100% - #{ $grid-unit-50 * 2 }); /* match full-screen `Modal` height */
+			max-height: none;
 		}
 	}