diff options
| author | 2024-04-30 14:47:08 -0700 | |
|---|---|---|
| committer | 2024-05-06 16:12:20 -0700 | |
| commit | 1d01102df43f30a50f585968bb8eac200dac72bf (patch) | |
| tree | fd6650dac9f2cb825c2c73e04bed408cbb4688c5 | |
| parent | a67cb4812144efcd5d91293f935c4055f720ac52 (diff) | |
Update Preview and Empty State with materialColor
Colors for text and background in resolver_empty_state
as well as content_preview_file_text do not match the
colors used in chooser_grid_preview_text.xml. This
results in the mixing of DeviceDefault colors with
Material colors, which could show black-on-black or
white-on-white.
Update them to all use the OnSurface colors to be
consistent with chooser_grid_preview_text.xml.
Bug: 338107805
Test: manual - see hsv in issue
Change-Id: Id24d8c66cf2b7985ee6de4ac4549a14fceea800e
| -rw-r--r-- | java/res/layout/chooser_grid_preview_files_text.xml | 1 | ||||
| -rw-r--r-- | java/res/layout/resolver_empty_states.xml | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/java/res/layout/chooser_grid_preview_files_text.xml b/java/res/layout/chooser_grid_preview_files_text.xml index 2756e800..65c62f82 100644 --- a/java/res/layout/chooser_grid_preview_files_text.xml +++ b/java/res/layout/chooser_grid_preview_files_text.xml @@ -53,6 +53,7 @@ android:maxLines="@integer/text_preview_lines" android:ellipsize="end" android:linksClickable="false" + android:textColor="?androidprv:attr/materialColorOnSurfaceVariant" android:textAppearance="@style/TextAppearance.ChooserDefault"/> </LinearLayout> diff --git a/java/res/layout/resolver_empty_states.xml b/java/res/layout/resolver_empty_states.xml index 922fca13..0cf6e955 100644 --- a/java/res/layout/resolver_empty_states.xml +++ b/java/res/layout/resolver_empty_states.xml @@ -84,6 +84,7 @@ android:layout_width="match_parent" android:layout_height="wrap_content" android:text="@string/noApplications" + android:textColor="?androidprv:attr/materialColorOnSurfaceVariant" android:padding="@dimen/chooser_edge_margin_normal" android:gravity="center"/> </RelativeLayout> |