From 929539eb864822d669265b142bdcc49b6cf8ea6b Mon Sep 17 00:00:00 2001 From: calderwoodra Date: Sat, 19 May 2018 00:05:02 -0700 Subject: Implement a global theme to be easily configured in Dialer. deleted several colors and unified them across the app migrated several alert dialogs to support alert dialogs added many todos migrated several tests from GoogleRobolectricTestRunner to RobolectricTestRunner Because of the test migration: - moved dialpad theme attributes into dialpad/theme - moved incall ui theme attributes into incallui/theme Bug: 79883035 Test: tap PiperOrigin-RevId: 197246477 Change-Id: Ifc534793bc32757bbbf2007a7c40287c8d0817ad --- java/com/android/dialer/app/AndroidManifest.xml | 6 +- .../app/calllog/BlockReportSpamListener.java | 2 +- .../android/dialer/app/calllog/CallLogAdapter.java | 5 +- .../app/manifests/activities/AndroidManifest.xml | 4 +- .../app/res/color/settings_text_color_primary.xml | 23 --- .../res/color/settings_text_color_secondary.xml | 23 --- .../app/res/drawable/ic_voicemail_seek_handle.xml | 2 +- .../drawable/selectable_primary_flat_button.xml | 2 +- .../dialer/app/res/layout/call_log_activity.xml | 2 +- .../dialer/app/res/layout/call_log_fragment.xml | 2 +- .../dialer/app/res/layout/call_log_list_item.xml | 2 +- .../dialer/app/res/layout/dialtacts_activity.xml | 2 +- .../dialer/app/res/layout/keyguard_preview.xml | 2 +- .../dialer/app/res/layout/lists_fragment.xml | 4 +- .../dialer/app/res/layout/search_edittext.xml | 2 +- .../app/res/layout/voicemail_playback_layout.xml | 2 +- .../android/dialer/app/res/values-v27/styles.xml | 4 +- java/com/android/dialer/app/res/values/colors.xml | 11 +- java/com/android/dialer/app/res/values/styles.xml | 162 ++++----------------- 19 files changed, 52 insertions(+), 210 deletions(-) delete mode 100644 java/com/android/dialer/app/res/color/settings_text_color_primary.xml delete mode 100644 java/com/android/dialer/app/res/color/settings_text_color_secondary.xml (limited to 'java/com/android/dialer/app') diff --git a/java/com/android/dialer/app/AndroidManifest.xml b/java/com/android/dialer/app/AndroidManifest.xml index 3536ecdc5..190f098a5 100644 --- a/java/com/android/dialer/app/AndroidManifest.xml +++ b/java/com/android/dialer/app/AndroidManifest.xml @@ -58,9 +58,9 @@ android:minSdkVersion="24" android:targetSdkVersion="28"/> - + - + android:theme="@style/DialtactsTheme"> diff --git a/java/com/android/dialer/app/calllog/BlockReportSpamListener.java b/java/com/android/dialer/app/calllog/BlockReportSpamListener.java index 987615f5f..865cb81cf 100644 --- a/java/com/android/dialer/app/calllog/BlockReportSpamListener.java +++ b/java/com/android/dialer/app/calllog/BlockReportSpamListener.java @@ -16,9 +16,9 @@ package com.android.dialer.app.calllog; -import android.app.FragmentManager; import android.content.Context; import android.support.annotation.NonNull; +import android.support.v4.app.FragmentManager; import android.support.v7.widget.RecyclerView; import android.view.View; import com.android.dialer.blocking.FilteredNumberAsyncQueryHandler; diff --git a/java/com/android/dialer/app/calllog/CallLogAdapter.java b/java/com/android/dialer/app/calllog/CallLogAdapter.java index 450e5eaa7..143af367a 100644 --- a/java/com/android/dialer/app/calllog/CallLogAdapter.java +++ b/java/com/android/dialer/app/calllog/CallLogAdapter.java @@ -36,6 +36,7 @@ import android.support.annotation.Nullable; import android.support.annotation.VisibleForTesting; import android.support.annotation.WorkerThread; import android.support.v7.app.AlertDialog; +import android.support.v7.app.AppCompatActivity; import android.support.v7.widget.RecyclerView; import android.support.v7.widget.RecyclerView.ViewHolder; import android.telecom.PhoneAccountHandle; @@ -223,7 +224,7 @@ public class CallLogAdapter extends GroupingListAdapter private void showDeleteSelectedItemsDialog() { SparseArray voicemailsToDeleteOnConfirmation = selectedItems.clone(); - new AlertDialog.Builder(activity, R.style.AlertDialogCustom) + new AlertDialog.Builder(activity) .setCancelable(true) .setTitle( activity @@ -557,7 +558,7 @@ public class CallLogAdapter extends GroupingListAdapter new BlockReportSpamListener( this.activity, this.activity.findViewById(R.id.call_log_fragment_root), - ((Activity) this.activity).getFragmentManager(), + ((AppCompatActivity) this.activity).getSupportFragmentManager(), this, this.filteredNumberAsyncQueryHandler); setHasStableIds(true); diff --git a/java/com/android/dialer/app/manifests/activities/AndroidManifest.xml b/java/com/android/dialer/app/manifests/activities/AndroidManifest.xml index 31960c979..edf0f1c8c 100644 --- a/java/com/android/dialer/app/manifests/activities/AndroidManifest.xml +++ b/java/com/android/dialer/app/manifests/activities/AndroidManifest.xml @@ -19,7 +19,7 @@ - + - + diff --git a/java/com/android/dialer/app/res/color/settings_text_color_primary.xml b/java/com/android/dialer/app/res/color/settings_text_color_primary.xml deleted file mode 100644 index ba259088a..000000000 --- a/java/com/android/dialer/app/res/color/settings_text_color_primary.xml +++ /dev/null @@ -1,23 +0,0 @@ - - - - - - - diff --git a/java/com/android/dialer/app/res/color/settings_text_color_secondary.xml b/java/com/android/dialer/app/res/color/settings_text_color_secondary.xml deleted file mode 100644 index 2f7899272..000000000 --- a/java/com/android/dialer/app/res/color/settings_text_color_secondary.xml +++ /dev/null @@ -1,23 +0,0 @@ - - - - - - - diff --git a/java/com/android/dialer/app/res/drawable/ic_voicemail_seek_handle.xml b/java/com/android/dialer/app/res/drawable/ic_voicemail_seek_handle.xml index ffdd64edc..00b23b84d 100644 --- a/java/com/android/dialer/app/res/drawable/ic_voicemail_seek_handle.xml +++ b/java/com/android/dialer/app/res/drawable/ic_voicemail_seek_handle.xml @@ -16,5 +16,5 @@ --> + android:tint="?android:attr/colorPrimary"> \ No newline at end of file diff --git a/java/com/android/dialer/app/res/drawable/selectable_primary_flat_button.xml b/java/com/android/dialer/app/res/drawable/selectable_primary_flat_button.xml index 47d1152db..2a393a570 100644 --- a/java/com/android/dialer/app/res/drawable/selectable_primary_flat_button.xml +++ b/java/com/android/dialer/app/res/drawable/selectable_primary_flat_button.xml @@ -25,7 +25,7 @@ - + \ No newline at end of file diff --git a/java/com/android/dialer/app/res/layout/call_log_activity.xml b/java/com/android/dialer/app/res/layout/call_log_activity.xml index a5e532698..d9d40a1a0 100644 --- a/java/com/android/dialer/app/res/layout/call_log_activity.xml +++ b/java/com/android/dialer/app/res/layout/call_log_activity.xml @@ -18,7 +18,7 @@ android:id="@+id/calllog_frame" android:layout_width="match_parent" android:layout_height="match_parent" - android:background="@color/background_dialer_light" + android:background="?android:attr/colorBackgroundFloating" android:orientation="vertical"> diff --git a/java/com/android/dialer/app/res/layout/call_log_list_item.xml b/java/com/android/dialer/app/res/layout/call_log_list_item.xml index d1111103e..23aeda3da 100644 --- a/java/com/android/dialer/app/res/layout/call_log_list_item.xml +++ b/java/com/android/dialer/app/res/layout/call_log_list_item.xml @@ -143,7 +143,7 @@ android:layout_height="wrap_content" android:layout_marginTop="@dimen/call_log_call_account_margin_bottom" android:layout_marginEnd="@dimen/call_log_icon_margin" - android:textColor="?android:textColorSecondary" + android:textColor="?android:attr/textColorSecondary" android:textSize="@dimen/call_log_detail_text_size" android:visibility="gone" android:maxLines="1"/> diff --git a/java/com/android/dialer/app/res/layout/dialtacts_activity.xml b/java/com/android/dialer/app/res/layout/dialtacts_activity.xml index 1627844d6..cf6591b63 100644 --- a/java/com/android/dialer/app/res/layout/dialtacts_activity.xml +++ b/java/com/android/dialer/app/res/layout/dialtacts_activity.xml @@ -19,7 +19,7 @@ android:id="@+id/dialtacts_mainlayout" android:layout_width="match_parent" android:layout_height="match_parent" - android:background="@color/background_dialer_light" + android:background="?android:attr/colorBackgroundFloating" android:clipChildren="false" android:clipToPadding="false" android:focusable="true" diff --git a/java/com/android/dialer/app/res/layout/keyguard_preview.xml b/java/com/android/dialer/app/res/layout/keyguard_preview.xml index eab98d261..e5207ac8b 100644 --- a/java/com/android/dialer/app/res/layout/keyguard_preview.xml +++ b/java/com/android/dialer/app/res/layout/keyguard_preview.xml @@ -23,7 +23,7 @@ + android:background="?android:attr/colorPrimaryDark"/> diff --git a/java/com/android/dialer/app/res/layout/search_edittext.xml b/java/com/android/dialer/app/res/layout/search_edittext.xml index 5fc492d29..8a2cfd483 100644 --- a/java/com/android/dialer/app/res/layout/search_edittext.xml +++ b/java/com/android/dialer/app/res/layout/search_edittext.xml @@ -31,7 +31,7 @@ android:id="@+id/search_box_collapsed" android:layout_width="match_parent" android:layout_height="match_parent" - android:background="?android:selectableItemBackground" + android:background="?android:attr/selectableItemBackground" android:gravity="center_vertical"> - diff --git a/java/com/android/dialer/app/res/values/colors.xml b/java/com/android/dialer/app/res/values/colors.xml index e79c5c5ed..2b8d63ac0 100644 --- a/java/com/android/dialer/app/res/values/colors.xml +++ b/java/com/android/dialer/app/res/values/colors.xml @@ -15,13 +15,8 @@ --> - @color/dialer_theme_color + ?android:attr/colorPrimary - - @color/dialer_primary_text_color - - @color/dialer_secondary_text_color - #aaaaaa #ffffff @@ -50,7 +45,7 @@ #cecece - @color/dialer_theme_color + ?android:attr/colorPrimary #f9f9f9 @@ -76,7 +71,7 @@ #616161 #FFFFFF #F44336 - @color/dialer_theme_color + ?android:attr/colorPrimary #E0E0E0 #bdbdbd diff --git a/java/com/android/dialer/app/res/values/styles.xml b/java/com/android/dialer/app/res/values/styles.xml index 869f1276d..54d18f103 100644 --- a/java/com/android/dialer/app/res/values/styles.xml +++ b/java/com/android/dialer/app/res/values/styles.xml @@ -15,134 +15,18 @@ ~ limitations under the License --> - - - - - - - - - + - - - - - - - - - - - - - -- cgit v1.2.3