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

Commit a229cd03 authored by Matt Casey's avatar Matt Casey
Browse files

Separate share icon color from button text. 

Different background, doesn't need the textColorPrimaryInverse
workaround.

Bug: 189114751
Test: Visual verification in light and dark themes.
Change-Id: I5e3f97ad5e2427c2eb17bc3717a7ad0a1f6e3260
parent d772c72a

packages/SystemUI/res/layout/long_screenshot.xml

+2 −2
Original line number Diff line number Diff line
@@ -33,7 +33,7 @@ 
        android:layout_marginStart="8dp"

        android:layout_marginTop="4dp"

        android:backgroundTint="?androidprv:attr/colorAccentSecondary"

        android:textColor="?android:textColorPrimary"

        android:textColor="?android:textColorSecondary"

        app:layout_constraintStart_toStartOf="parent"

        app:layout_constraintTop_toTopOf="parent"

        app:layout_constraintBottom_toTopOf="@id/preview" />
@@ -117,7 +117,7 @@ 
        android:backgroundTint="?androidprv:attr/colorAccentSecondary"

        android:src="@drawable/ic_screenshot_edit"

        android:contentDescription="@string/screenshot_edit_label"

        android:tint="?android:textColorPrimary"

        android:tint="?android:textColorSecondary"

        android:padding="16dp"

        android:scaleType="fitCenter"

        app:layout_constraintBottom_toBottomOf="parent"

packages/SystemUI/res/values-night/styles.xml

+1 −1
Original line number Diff line number Diff line
@@ -35,7 +35,7 @@ 
        <item name="android:windowLightStatusBar">false</item>

        <item name="android:windowLightNavigationBar">false</item>

        <item name="android:navigationBarColor">?android:attr/colorBackgroundFloating</item>

        <item name="android:textColorPrimary">?android:attr/textColorPrimaryInverse</item>

        <item name="android:textColorSecondary">?android:attr/textColorPrimaryInverse</item>

    </style>



    <style name="Screenshot" parent="@android:style/Theme.DeviceDefault.DayNight">