Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d06608d6 authored by Jeff DeCew's avatar Jeff DeCew
Browse files

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
parent 39f3d9bb

packages/SystemUI/res/values/dimens.xml

+1 −1
Original line number Diff line number Diff line
@@ -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>