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

Commit b2f6066a authored by Mady Mellor's avatar Mady Mellor
Browse files

Slight adjustments to bubble bar education view values 

Need to apply the padding to the text view, BubblePopupViewExt sets
its own padding to account for the carrot so the bottom padding
was getting overridden. Setting it on the text view ensures it's
correct.

Also missed setting one colour!

Flag: com.android.wm.shell.enable_bubble_bar
Test: manual - enable user education and inspect the views
Bug: 372516663
Change-Id: I3db8c4fcaa0006f49af26ce0359ec58ab08525c3
parent 7c04f065

libs/WindowManager/Shell/res/layout/bubble_bar_manage_education.xml

+2 −2
Original line number Diff line number Diff line
@@ -22,7 +22,6 @@ 
    android:layout_gravity="center_horizontal"

    android:layout_marginTop="@dimen/bubble_popup_margin_top"

    android:layout_marginHorizontal="@dimen/bubble_popup_margin_horizontal"

    android:layout_marginBottom="@dimen/bubble_popup_margin_bottom"

    android:elevation="@dimen/bubble_popup_elevation"

    android:gravity="center_horizontal"

    android:orientation="vertical">
@@ -30,7 +29,7 @@ 
    <ImageView

        android:layout_width="@dimen/bubble_popup_icon_size"

        android:layout_height="@dimen/bubble_popup_icon_size"

        android:tint="?androidprv:attr/materialColorPrimary"

        android:tint="?androidprv:attr/materialColorOutline"

        android:contentDescription="@null"

        android:src="@drawable/pip_ic_settings"/>
@@ -49,6 +48,7 @@ 
        android:layout_width="wrap_content"

        android:layout_height="wrap_content"

        android:layout_marginTop="@dimen/bubble_popup_text_margin"

        android:paddingBottom="@dimen/bubble_popup_padding_bottom"

        android:maxWidth="@dimen/bubble_popup_content_max_width"

        android:textAppearance="@android:style/TextAppearance.DeviceDefault"

        android:textColor="?androidprv:attr/materialColorOnSurfaceVariant"