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

Commit 0b8a6fe5 authored by Tony's avatar Tony
Browse files

Visual updates for popup 

- Update some dimens
  - Increase left padding for shortcuts
  - Adjust arrow and notification ellipsis offset accordingly
  - Align shortcut dividers to be same width as text
  - Notification text 14sp instead of 15sp
  - Increase width of popup by 12dp
- Shortcuts text 87% black instead of 54%
- Arrow has same color as attached side, so takes on header
  gray instead of white when appropriate

Bug: 37753827
Change-Id: I0844b85995f22218a3b7e605e1653cceaed6cda1
parent 9a122710

res/layout/deep_shortcut.xml

+2 −1
Original line number Diff line number Diff line
@@ -26,11 +26,12 @@ 
        android:background="?android:attr/selectableItemBackground"

        android:gravity="start|center_vertical"

        android:textAlignment="viewStart"

        android:paddingStart="@dimen/bg_popup_item_height"

        android:paddingStart="@dimen/deep_shortcuts_text_padding_start"

        android:paddingEnd="@dimen/popup_padding_end"

        android:drawableEnd="@drawable/deep_shortcuts_drag_handle"

        android:drawablePadding="@dimen/deep_shortcut_drawable_padding"

        android:textSize="14sp"

        android:textColor="?android:attr/textColorPrimary"

        android:fontFamily="sans-serif"

        launcher:layoutHorizontal="true"

        launcher:iconDisplay="shortcut_popup"

res/layout/notification.xml

+2 −2
Original line number Diff line number Diff line
@@ -35,7 +35,7 @@ 
            android:layout_height="@dimen/notification_header_height"

            android:paddingStart="@dimen/notification_padding_start"

            android:paddingEnd="@dimen/notification_padding_end"

            android:background="@color/notification_header_background_color"

            android:background="@color/popup_header_background_color"

            android:elevation="@dimen/notification_elevation">

            <TextView

                android:id="@+id/notification_text"
@@ -67,7 +67,7 @@ 
            android:id="@+id/divider"

            android:layout_width="match_parent"

            android:layout_height="@dimen/popup_item_divider_height"

            android:background="@color/divider_color"

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

            android:layout_below="@id/main_view"/>



        <include layout="@layout/notification_footer"

res/layout/notification_footer.xml

+1 −1
Original line number Diff line number Diff line
@@ -22,7 +22,7 @@ 
    android:elevation="@dimen/notification_elevation"

    android:clipChildren="false"

    android:layout_gravity="center_vertical"

    android:background="@color/notification_background_color">

    android:background="@color/popup_background_color">



    <LinearLayout

        android:id="@+id/icon_row"

res/layout/notification_main.xml

+1 −1
Original line number Diff line number Diff line
@@ -28,7 +28,7 @@ 
        android:layout_height="match_parent"

        android:orientation="vertical"

        android:gravity="center_vertical"

        android:background="@color/notification_background_color"

        android:background="@color/popup_background_color"

        android:paddingStart="@dimen/notification_padding_start"

        android:paddingEnd="@dimen/notification_main_text_padding_end">

        <TextView

res/layout/system_shortcut.xml

+2 −1
Original line number Diff line number Diff line
@@ -26,9 +26,10 @@ 
        android:background="?android:attr/selectableItemBackground"

        android:gravity="start|center_vertical"

        android:textAlignment="viewStart"

        android:paddingStart="@dimen/bg_popup_item_height"

        android:paddingStart="@dimen/deep_shortcuts_text_padding_start"

        android:paddingEnd="@dimen/popup_padding_end"

        android:textSize="14sp"

        android:textColor="?android:attr/textColorPrimary"

        android:fontFamily="sans-serif"

        launcher:iconDisplay="shortcut_popup"

        launcher:layoutHorizontal="true" />