diff --git a/packages/SystemUI/res/layout/notification_conversation_info.xml b/packages/SystemUI/res/layout/notification_conversation_info.xml index 3bd7e04f20940..49d525fc189a1 100644 --- a/packages/SystemUI/res/layout/notification_conversation_info.xml +++ b/packages/SystemUI/res/layout/notification_conversation_info.xml @@ -84,34 +84,21 @@ android:layout_weight="1" style="@style/TextAppearance.NotificationImportanceChannel"/> - - - - - + style="@style/TextAppearance.NotificationImportanceChannelGroup" + android:ellipsize="end" + android:textDirection="locale" + android:maxLines="1"/> + diff --git a/packages/SystemUI/res/layout/notification_info.xml b/packages/SystemUI/res/layout/notification_info.xml index af5a8f4d3c23b..870deacd87ae0 100644 --- a/packages/SystemUI/res/layout/notification_info.xml +++ b/packages/SystemUI/res/layout/notification_info.xml @@ -58,35 +58,21 @@ android:layout_height="wrap_content" android:textDirection="locale" style="@style/TextAppearance.NotificationImportanceChannel"/> - - - - - + style="@style/TextAppearance.NotificationImportanceChannelGroup" + android:ellipsize="end" + android:textDirection="locale" + android:maxLines="1"/> + diff --git a/packages/SystemUI/res/layout/partial_conversation_info.xml b/packages/SystemUI/res/layout/partial_conversation_info.xml index a261114fa0f21..b34822291815b 100644 --- a/packages/SystemUI/res/layout/partial_conversation_info.xml +++ b/packages/SystemUI/res/layout/partial_conversation_info.xml @@ -46,7 +46,6 @@ android:layout_weight="1" android:layout_width="0dp" android:orientation="vertical" - android:layout_height="wrap_content" android:minHeight="@dimen/notification_guts_conversation_icon_size" android:layout_centerVertical="true" @@ -78,34 +77,21 @@ style="@style/TextAppearance.NotificationImportanceChannel"/> - - - - - + style="@style/TextAppearance.NotificationImportanceChannelGroup" + android:ellipsize="end" + android:textDirection="locale" + android:maxLines="1"/> + diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationConversationInfo.java b/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationConversationInfo.java index 9befa313edd89..2e7b175870857 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationConversationInfo.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationConversationInfo.java @@ -368,14 +368,11 @@ public class NotificationConversationInfo extends LinearLayout implements } } TextView groupNameView = findViewById(R.id.group_name); - View groupDivider = findViewById(R.id.group_divider); if (groupName != null) { groupNameView.setText(groupName); groupNameView.setVisibility(VISIBLE); - groupDivider.setVisibility(VISIBLE); } else { groupNameView.setVisibility(GONE); - groupDivider.setVisibility(GONE); } } diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationInfo.java b/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationInfo.java index 334599930b63c..f434fbbcd9166 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationInfo.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationInfo.java @@ -375,14 +375,11 @@ public class NotificationInfo extends LinearLayout implements NotificationGuts.G } } TextView groupNameView = findViewById(R.id.group_name); - View divider = findViewById(R.id.group_divider); if (groupName != null) { groupNameView.setText(groupName); groupNameView.setVisibility(VISIBLE); - divider.setVisibility(VISIBLE); } else { groupNameView.setVisibility(GONE); - divider.setVisibility(GONE); } } diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/PartialConversationInfo.java b/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/PartialConversationInfo.java index ea059cbcf3e14..eb28c58d95b74 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/PartialConversationInfo.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/PartialConversationInfo.java @@ -269,14 +269,11 @@ public class PartialConversationInfo extends LinearLayout implements } } TextView groupNameView = findViewById(R.id.group_name); - View groupDivider = findViewById(R.id.group_divider); if (groupName != null) { groupNameView.setText(groupName); groupNameView.setVisibility(VISIBLE); - groupDivider.setVisibility(VISIBLE); } else { groupNameView.setVisibility(GONE); - groupDivider.setVisibility(GONE); } } diff --git a/packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/row/NotificationConversationInfoTest.java b/packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/row/NotificationConversationInfoTest.java index 0272028e62fc0..b39de34f6c638 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/row/NotificationConversationInfoTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/row/NotificationConversationInfoTest.java @@ -336,7 +336,6 @@ public class NotificationConversationInfoTest extends SysuiTestCase { assertTrue(textView.getText().toString().contains(group.getName())); assertEquals(VISIBLE, mNotificationInfo.findViewById(R.id.header).getVisibility()); assertEquals(VISIBLE, textView.getVisibility()); - assertEquals(VISIBLE, mNotificationInfo.findViewById(R.id.group_divider).getVisibility()); } @Test @@ -361,7 +360,6 @@ public class NotificationConversationInfoTest extends SysuiTestCase { final TextView textView = mNotificationInfo.findViewById(R.id.group_name); assertEquals(VISIBLE, mNotificationInfo.findViewById(R.id.header).getVisibility()); assertEquals(GONE, textView.getVisibility()); - assertEquals(GONE, mNotificationInfo.findViewById(R.id.group_divider).getVisibility()); } @Test diff --git a/packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/row/NotificationInfoTest.java b/packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/row/NotificationInfoTest.java index 8ee86a237e320..6bf60721cd8ed 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/row/NotificationInfoTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/row/NotificationInfoTest.java @@ -236,8 +236,6 @@ public class NotificationInfoTest extends SysuiTestCase { true); final TextView nameView = mNotificationInfo.findViewById(R.id.delegate_name); assertEquals(GONE, nameView.getVisibility()); - final TextView dividerView = mNotificationInfo.findViewById(R.id.group_divider); - assertEquals(GONE, dividerView.getVisibility()); } @Test @@ -288,8 +286,6 @@ public class NotificationInfoTest extends SysuiTestCase { true); final TextView groupNameView = mNotificationInfo.findViewById(R.id.group_name); assertEquals(GONE, groupNameView.getVisibility()); - final TextView dividerView = mNotificationInfo.findViewById(R.id.group_divider); - assertEquals(GONE, dividerView.getVisibility()); } @Test @@ -317,8 +313,6 @@ public class NotificationInfoTest extends SysuiTestCase { final TextView groupNameView = mNotificationInfo.findViewById(R.id.group_name); assertEquals(View.VISIBLE, groupNameView.getVisibility()); assertEquals("Test Group Name", groupNameView.getText()); - final TextView dividerView = mNotificationInfo.findViewById(R.id.group_divider); - assertEquals(View.VISIBLE, dividerView.getVisibility()); } @Test diff --git a/packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/row/PartialConversationInfoTest.java b/packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/row/PartialConversationInfoTest.java index 545b59a4556ac..e56ef5b92f870 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/row/PartialConversationInfoTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/row/PartialConversationInfoTest.java @@ -259,8 +259,6 @@ public class PartialConversationInfoTest extends SysuiTestCase { false); final TextView nameView = mInfo.findViewById(R.id.delegate_name); assertEquals(GONE, nameView.getVisibility()); - final TextView dividerView = mInfo.findViewById(R.id.group_divider); - assertEquals(GONE, dividerView.getVisibility()); } @Test @@ -305,8 +303,6 @@ public class PartialConversationInfoTest extends SysuiTestCase { false); final TextView groupNameView = mInfo.findViewById(R.id.group_name); assertEquals(GONE, groupNameView.getVisibility()); - final TextView dividerView = mInfo.findViewById(R.id.group_divider); - assertEquals(GONE, dividerView.getVisibility()); } @Test @@ -331,8 +327,6 @@ public class PartialConversationInfoTest extends SysuiTestCase { final TextView groupNameView = mInfo.findViewById(R.id.group_name); assertEquals(View.VISIBLE, groupNameView.getVisibility()); assertEquals("Test Group Name", groupNameView.getText()); - final TextView dividerView = mInfo.findViewById(R.id.group_divider); - assertEquals(View.VISIBLE, dividerView.getVisibility()); } @Test