From 6ce574731f14b5c675f5ddeadd2a93f4f105c825 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Joy=20Serqui=C3=B1a?=
 <44146839+essjay05@users.noreply.github.com>
Date: Thu, 10 Oct 2024 15:29:31 -0700
Subject: [PATCH] fix(material/dialog): updates dialog max-height in landscape
 (#29853)

* fix(material/dialog): updates dialog max-height in landscape

Updates Angular Component Dialog component to have a max-height of
95vh which gives it a calculable height for its contents to adjust
on smaller screens or in landscape mode so that the action buttons
are able to be accessed by scrolling rather than the dialog window
cutting them off.

Fixes b/323588333

* refactor(material/dialog): reverted max-height to inherit

Updates previous fix to revert max-height back to inherit.

Fixes b/323588333

(cherry picked from commit da9cb714e92dc2fc997043bbca33004910763998)
---
 src/material/dialog/dialog.scss | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/material/dialog/dialog.scss b/src/material/dialog/dialog.scss
index 9487190311db..5e98a084ad0f 100644
--- a/src/material/dialog/dialog.scss
+++ b/src/material/dialog/dialog.scss
@@ -11,6 +11,11 @@ $mat-dialog-content-max-height: 65vh !default;
 // don't expose this value as variable.
 $mat-dialog-button-horizontal-margin: 8px !default;
 
+// Dialog container max height. This has been given a default value so the
+// flex-children can be calculated and not overflow on smaller screens.
+// Fixes b/323588333
+$mat-dialog-container-max-height: 95vh !default;
+
 // Whether to emit fallback values for the structural styles. Previously MDC was emitting
 // paddings in the static styles which meant that users would get them even if they didn't
 // include the `dialog-base`. Eventually we should clean up the usages of this flag.