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

Commit 3dd023f3 authored by Julia Reynolds's avatar Julia Reynolds
Browse files

UX updates for notif history 

- no relative time
- fix background color in dark theme
- fix text alignment in RTL

Test: manual
Bug: 160061165
Bug: 160388261
Bug: 154795795
Change-Id: I7d4c5a5f81ce5b5ce43fdaa3ee99184906753a76
parent f04c1674

res/layout/notification_history_app_layout.xml

+2 −1
Original line number Diff line number Diff line
@@ -55,7 +55,8 @@ 
                android:layout_height="wrap_content"

                android:layout_width="wrap_content"

                android:paddingStart="6dp"

                android:layout_gravity="center_vertical"

                android:layout_gravity="start|center_vertical"

                android:textDirection="locale"

                android:paddingTop="8dp"

                android:textAppearance="@*android:style/TextAppearance.DeviceDefault.Notification.Title"/>

        </LinearLayout>

res/layout/notification_sbn_log_row.xml

+2 −2
Original line number Diff line number Diff line
@@ -130,6 +130,6 @@ 
    <View

        android:id="@+id/divider"

        android:layout_width="match_parent"

        android:layout_height="0.5dp"

        android:background="@color/material_grey_300" />

        android:layout_height="wrap_content"

        android:background="?android:attr/listDivider" />

</LinearLayout>
 
 No newline at end of file

res/values-night/colors.xml

+1 −0
Original line number Diff line number Diff line
@@ -27,6 +27,7 @@ 
    <color name="notification_importance_selection_bg">@*android:color/material_grey_800</color>

    <color name="notification_importance_button_selected">#AECBFA</color> <!-- material blue 200 -->

    <color name="notification_importance_button_unselected">#5F6368</color>

    <color name="notification_history_background">#202124</color>

    <color name="face_intro_outline">?android:attr/colorAccent</color>

    <!-- Palette list preference colors. -->

    <color name="palette_list_gradient_background">@android:color/black</color>

res/values/colors.xml

+1 −0
Original line number Diff line number Diff line
@@ -129,6 +129,7 @@ 
    <color name="notification_importance_button_unselected">#DADCE0</color>

    <color name="notification_importance_button_selected">#1967D2</color> <!-- material blue 700 -->

    <color name="notification_importance_selection_bg">#FFFFFF</color>

    <color name="notification_history_background">?android:attr/colorBackgroundFloating</color>



    <!-- launcher icon color -->

    <color name="icon_launcher_setting_color">@*android:color/accent_device_default_light</color>

src/com/android/settings/notification/history/NotificationHistoryViewHolder.java

+0 −1
Original line number Diff line number Diff line
@@ -33,7 +33,6 @@ public class NotificationHistoryViewHolder extends RecyclerView.ViewHolder { 
    NotificationHistoryViewHolder(View itemView) {

        super(itemView);

        mTime = itemView.findViewById(R.id.timestamp);

        mTime.setShowRelativeTime(true);

        mTitle = itemView.findViewById(R.id.title);

        mSummary = itemView.findViewById(R.id.text);

    }