From 6fb9669e441266b4bcc694b920ca59a3f158c815 Mon Sep 17 00:00:00 2001
From: Julia Reynolds <juliacr@google.com>
Date: Wed, 1 Apr 2020 12:05:25 -0400
Subject: [PATCH] Some more a11y fixes

Min height, and announcing button changes

Test: manual
Fixes: 152729661
Fixes: 152728211
Change-Id: I886ce046da3dd5b2b93673aee0cd5bdc37d5fc48
---
 res/layout/notification_history_log_row.xml                      | 1 +
 res/layout/notification_sbn_log_row.xml                          | 1 +
 .../notification/history/NotificationHistoryActivity.java        | 1 +
 3 files changed, 3 insertions(+)

diff --git a/res/layout/notification_history_log_row.xml b/res/layout/notification_history_log_row.xml
index c77b005b054..d2d7cd60e6e 100644
--- a/res/layout/notification_history_log_row.xml
+++ b/res/layout/notification_history_log_row.xml
@@ -26,6 +26,7 @@
     <LinearLayout
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
+        android:minHeight="48dp"
         android:orientation="vertical"
         android:layout_marginBottom="6dp">
         <LinearLayout
diff --git a/res/layout/notification_sbn_log_row.xml b/res/layout/notification_sbn_log_row.xml
index 5fa1b5d8c20..3eb6796ed3f 100644
--- a/res/layout/notification_sbn_log_row.xml
+++ b/res/layout/notification_sbn_log_row.xml
@@ -104,6 +104,7 @@
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
         android:orientation="vertical"
+        android:minHeight="48dp"
         android:layout_marginStart="30dp"
         android:layout_marginBottom="6dp"
         >
diff --git a/src/com/android/settings/notification/history/NotificationHistoryActivity.java b/src/com/android/settings/notification/history/NotificationHistoryActivity.java
index 21aecdbab39..8d7695ff3d8 100644
--- a/src/com/android/settings/notification/history/NotificationHistoryActivity.java
+++ b/src/com/android/settings/notification/history/NotificationHistoryActivity.java
@@ -90,6 +90,7 @@ public class NotificationHistoryActivity extends Activity {
                     expand.setContentDescription(container.getVisibility() == View.VISIBLE
                             ? getString(R.string.condition_expand_hide)
                             : getString(R.string.condition_expand_show));
+                    expand.setAccessibilityLiveRegion(View.ACCESSIBILITY_LIVE_REGION_POLITE);
             });
 
             TextView label = viewForPackage.findViewById(R.id.label);