From d06608d6c9d1d69e0dec0c4cf8417263eebfcc04 Mon Sep 17 00:00:00 2001
From: Jeff DeCew <jeffdq@google.com>
Date: Mon, 26 Apr 2021 17:29:11 -0400
Subject: Increase the maximum height of notifications by 20%

This value is the exact height of 4 collapsed notifications + padding (88*4+2*3)

We are increasing the height of notifications because we've increased the font sizes, line caps, and paddings, reducing the amount of content that can be shown in the same area.  Specifically:
* BigPictureStyle's pictures were getting quite cramped, showing at almost 3:1 aspect ratio in some cases
* ConversationLayout for group chats have larger icons and paddings that reduced the number of messages visible to 2, but this change restores that to 3.

Fixes: 186456052
Test: manual
Change-Id: Iccade055de96ab1e860eec313e9e79b09c66c637
---
 packages/SystemUI/res/values/dimens.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/SystemUI/res/values/dimens.xml b/packages/SystemUI/res/values/dimens.xml
index fff4a1b74e56..2eb34fedb3e4 100644
--- a/packages/SystemUI/res/values/dimens.xml
+++ b/packages/SystemUI/res/values/dimens.xml
@@ -139,7 +139,7 @@
     <dimen name="notification_min_height_before_s">106dp</dimen>
 
     <!-- Height of a large notification in the status bar -->
-    <dimen name="notification_max_height">294dp</dimen>
+    <dimen name="notification_max_height">358dp</dimen>
 
     <!-- Height of a heads up notification in the status bar for legacy custom views -->
     <dimen name="notification_max_heads_up_height_legacy">128dp</dimen>
-- 
cgit v1.2.3-59-g8ed1b