From 4464ae54a32ec7dda3600d27aa5985898f50453a Mon Sep 17 00:00:00 2001 From: Laurent Date: Fri, 12 Jun 2020 15:09:46 +0200 Subject: [PATCH] More chip color + color refactoring --- .../handhistory/replayer/ReplayerView.kt | 2 +- .../modules/handhistory/replayer/TableDrawer.kt | 13 ++++++++----- .../main/res/color/bottom_navigation_item.xml | 4 ++-- .../drawable-xxhdpi/circle_player_color_1.xml | 2 +- app/src/main/res/drawable/ic_arrow_down.xml | 2 +- app/src/main/res/drawable/ic_arrow_right.xml | 2 +- app/src/main/res/drawable/ic_arrow_up.xml | 2 +- .../layout/bottom_sheet_double_edit_text.xml | 2 +- .../main/res/layout/bottom_sheet_edit_text.xml | 2 +- .../bottom_sheet_edit_text_multi_lines.xml | 2 +- app/src/main/res/layout/bottom_sheet_sum.xml | 2 +- .../main/res/layout/fragment_bottom_sheet.xml | 2 +- app/src/main/res/layout/fragment_replayer.xml | 12 ++++++------ .../main/res/layout/fragment_subscription.xml | 2 +- app/src/main/res/layout/row_content.xml | 2 +- app/src/main/res/layout/row_legend_default.xml | 2 +- app/src/main/res/layout/row_player.xml | 4 ++-- app/src/main/res/layout/row_player_image.xml | 2 +- app/src/main/res/layout/row_session_view.xml | 2 +- .../main/res/layout/row_stats_title_value.xml | 2 +- app/src/main/res/layout/row_title.xml | 2 +- app/src/main/res/layout/row_title_arrow.xml | 2 +- .../main/res/layout/row_title_icon_arrow.xml | 2 +- .../main/res/layout/row_title_value_action.xml | 2 +- .../main/res/layout/row_title_value_arrow.xml | 2 +- .../main/res/layout/row_transaction_view.xml | 2 +- app/src/main/res/values/colors.xml | 8 ++++---- app/src/main/res/values/styles.xml | 17 +++++++++++------ 28 files changed, 55 insertions(+), 47 deletions(-) diff --git a/app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/replayer/ReplayerView.kt b/app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/replayer/ReplayerView.kt index 4264ecac..5100d68e 100644 --- a/app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/replayer/ReplayerView.kt +++ b/app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/replayer/ReplayerView.kt @@ -41,7 +41,7 @@ class ReplayerView(context: Context, attrs: AttributeSet) : View(context, attrs) override fun onDraw(canvas: Canvas?) { super.onDraw(canvas) - Timber.d("ReplayerView > onDraw") +// Timber.d("ReplayerView > onDraw") canvas?.let { diff --git a/app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/replayer/TableDrawer.kt b/app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/replayer/TableDrawer.kt index 7cddc98a..f7f65e5a 100644 --- a/app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/replayer/TableDrawer.kt +++ b/app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/replayer/TableDrawer.kt @@ -12,24 +12,27 @@ import net.pokeranalytics.android.model.realm.handhistory.Card import net.pokeranalytics.android.ui.modules.handhistory.model.ComputedAction import net.pokeranalytics.android.util.RANDOM_PLAYER import net.pokeranalytics.android.util.extensions.formatted -import timber.log.Timber data class ChipColor(val fillColor: Int, val borderColor: Int) { companion object { val all: List by lazy { listOf( ChipColor(R.color.white, R.color.grey), + ChipColor(R.color.kaki_light, R.color.white_dark), ChipColor(R.color.green, R.color.white), ChipColor(R.color.player_color_2, R.color.chip_dash_color_2), ChipColor(R.color.player_color_3, R.color.chip_dash_color_3), - ChipColor(R.color.player_color_4, R.color.chip_dash_color_4), + ChipColor(R.color.player_color_9, R.color.chip_dash_color_9), ChipColor(R.color.player_color_5, R.color.chip_dash_color_5), - ChipColor(R.color.player_color_6, R.color.chip_dash_color_6), ChipColor(R.color.player_color_7, R.color.chip_dash_color_7), + ChipColor(R.color.player_color_6, R.color.chip_dash_color_6), ChipColor(R.color.player_color_8, R.color.chip_dash_color_8), - ChipColor(R.color.player_color_9, R.color.chip_dash_color_9), + ChipColor(R.color.player_color_4, R.color.chip_dash_color_4), ChipColor(R.color.chip_dash_color_7, R.color.player_color_2), - ChipColor(R.color.player_color_6, R.color.player_color_9) + ChipColor(R.color.chip_dash_color_4, R.color.player_color_3), + ChipColor(R.color.player_color_6, R.color.player_color_9), + ChipColor(R.color.player_color_8, R.color.player_color_2), + ChipColor(R.color.chip_dash_color_7, R.color.player_color_4) ) } diff --git a/app/src/main/res/color/bottom_navigation_item.xml b/app/src/main/res/color/bottom_navigation_item.xml index 5c3998df..30f4760c 100644 --- a/app/src/main/res/color/bottom_navigation_item.xml +++ b/app/src/main/res/color/bottom_navigation_item.xml @@ -2,7 +2,7 @@ - - + + \ No newline at end of file diff --git a/app/src/main/res/drawable-xxhdpi/circle_player_color_1.xml b/app/src/main/res/drawable-xxhdpi/circle_player_color_1.xml index 8cf572ee..38d1190f 100644 --- a/app/src/main/res/drawable-xxhdpi/circle_player_color_1.xml +++ b/app/src/main/res/drawable-xxhdpi/circle_player_color_1.xml @@ -5,7 +5,7 @@ + android:fillColor="@color/grey_light"/> diff --git a/app/src/main/res/drawable/ic_arrow_right.xml b/app/src/main/res/drawable/ic_arrow_right.xml index 19d1897e..91735d45 100644 --- a/app/src/main/res/drawable/ic_arrow_right.xml +++ b/app/src/main/res/drawable/ic_arrow_right.xml @@ -1,6 +1,6 @@ - diff --git a/app/src/main/res/drawable/ic_arrow_up.xml b/app/src/main/res/drawable/ic_arrow_up.xml index 59b4d185..22c45996 100644 --- a/app/src/main/res/drawable/ic_arrow_up.xml +++ b/app/src/main/res/drawable/ic_arrow_up.xml @@ -5,5 +5,5 @@ android:viewportHeight="24"> + android:fillColor="@color/grey_light"/> diff --git a/app/src/main/res/layout/bottom_sheet_double_edit_text.xml b/app/src/main/res/layout/bottom_sheet_double_edit_text.xml index 703d8f60..d8d0ce52 100644 --- a/app/src/main/res/layout/bottom_sheet_double_edit_text.xml +++ b/app/src/main/res/layout/bottom_sheet_double_edit_text.xml @@ -5,7 +5,7 @@ android:layout_width="match_parent" android:layout_height="wrap_content" android:orientation="vertical" - tools:background="@color/gray_darker"> + tools:background="@color/grey_darkest"> + tools:background="@color/grey_darkest"> + tools:background="@color/grey_darkest"> + tools:background="@color/grey_darkest"> + android:background="@color/grey_darker"> diff --git a/app/src/main/res/layout/row_content.xml b/app/src/main/res/layout/row_content.xml index 49c1bf77..8091edc5 100644 --- a/app/src/main/res/layout/row_content.xml +++ b/app/src/main/res/layout/row_content.xml @@ -12,7 +12,7 @@ android:id="@+id/foreground" android:layout_width="match_parent" android:layout_height="match_parent" - android:background="@color/gray_dark" + android:background="@color/grey_darker" android:foreground="?selectableItemBackground"> diff --git a/app/src/main/res/layout/row_player.xml b/app/src/main/res/layout/row_player.xml index 3a8582f1..ae805f9a 100644 --- a/app/src/main/res/layout/row_player.xml +++ b/app/src/main/res/layout/row_player.xml @@ -12,7 +12,7 @@ android:id="@+id/foreground" android:layout_width="match_parent" android:layout_height="wrap_content" - android:background="@color/gray_dark" + android:background="@color/grey_darker" android:foreground="?selectableItemBackground"> diff --git a/app/src/main/res/layout/row_player_image.xml b/app/src/main/res/layout/row_player_image.xml index 2a8a6c20..803c7b1f 100644 --- a/app/src/main/res/layout/row_player_image.xml +++ b/app/src/main/res/layout/row_player_image.xml @@ -3,7 +3,7 @@ xmlns:app="http://schemas.android.com/apk/res-auto" android:layout_width="match_parent" android:layout_height="wrap_content" - android:background="@color/gray_dark" + android:background="@color/grey_darker" android:foreground="?selectableItemBackground"> diff --git a/app/src/main/res/layout/row_stats_title_value.xml b/app/src/main/res/layout/row_stats_title_value.xml index 482095f6..f428a1ed 100644 --- a/app/src/main/res/layout/row_stats_title_value.xml +++ b/app/src/main/res/layout/row_stats_title_value.xml @@ -37,7 +37,7 @@ android:layout_width="24dp" android:layout_height="24dp" android:src="@drawable/ic_arrow_right" - android:tint="@color/gray_light" + android:tint="@color/grey_light" android:visibility="gone" app:layout_constraintBottom_toBottomOf="parent" app:layout_constraintEnd_toEndOf="@+id/guidelineEnd" diff --git a/app/src/main/res/layout/row_title.xml b/app/src/main/res/layout/row_title.xml index 8bd71ab6..9921a28e 100644 --- a/app/src/main/res/layout/row_title.xml +++ b/app/src/main/res/layout/row_title.xml @@ -12,7 +12,7 @@ android:id="@+id/foreground" android:layout_width="match_parent" android:layout_height="match_parent" - android:background="@color/gray_dark" + android:background="@color/grey_darker" android:foreground="?selectableItemBackground"> diff --git a/app/src/main/res/layout/row_title_icon_arrow.xml b/app/src/main/res/layout/row_title_icon_arrow.xml index bea82a99..c91106c5 100644 --- a/app/src/main/res/layout/row_title_icon_arrow.xml +++ b/app/src/main/res/layout/row_title_icon_arrow.xml @@ -37,7 +37,7 @@ android:layout_width="24dp" android:layout_height="24dp" android:src="@drawable/ic_arrow_right" - android:tint="@color/gray_light" + android:tint="@color/grey_light" app:layout_constraintBottom_toBottomOf="parent" app:layout_constraintEnd_toEndOf="@+id/guidelineEnd" app:layout_constraintTop_toTopOf="parent" /> diff --git a/app/src/main/res/layout/row_title_value_action.xml b/app/src/main/res/layout/row_title_value_action.xml index b43cf74f..54e80882 100644 --- a/app/src/main/res/layout/row_title_value_action.xml +++ b/app/src/main/res/layout/row_title_value_action.xml @@ -12,7 +12,7 @@ android:id="@+id/foreground" android:layout_width="match_parent" android:layout_height="match_parent" - android:background="@color/gray_dark" + android:background="@color/grey_darker" android:foreground="?selectableItemBackground"> diff --git a/app/src/main/res/layout/row_transaction_view.xml b/app/src/main/res/layout/row_transaction_view.xml index 3473977d..a4df34f0 100644 --- a/app/src/main/res/layout/row_transaction_view.xml +++ b/app/src/main/res/layout/row_transaction_view.xml @@ -85,7 +85,7 @@ android:layout_width="24dp" android:layout_height="24dp" android:src="@drawable/ic_arrow_right" - android:tint="@color/gray_light" + android:tint="@color/grey_light" app:layout_constraintBottom_toBottomOf="parent" app:layout_constraintEnd_toEndOf="@+id/guidelineEnd" app:layout_constraintTop_toTopOf="parent" /> diff --git a/app/src/main/res/values/colors.xml b/app/src/main/res/values/colors.xml index 313d7062..5314f613 100644 --- a/app/src/main/res/values/colors.xml +++ b/app/src/main/res/values/colors.xml @@ -10,10 +10,10 @@ #FFFFFF #30FFFFFF - #8AFFFFFF - #141414 - #1B1F1B - #808080 + #8AFFFFFF + #141414 + #1B1F1B + #808080 #e0e0e0 #40000000 diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml index 30f80198..ed2e4464 100644 --- a/app/src/main/res/values/styles.xml +++ b/app/src/main/res/values/styles.xml @@ -7,7 +7,7 @@ @color/colorPrimaryDark @color/colorAccent @color/green_transparent - @color/gray_dark + @color/grey_darker @color/colorPrimary @color/white @color/white @@ -51,7 +51,7 @@ @@ -62,7 +62,7 @@ @@ -91,7 +91,7 @@ @@ -330,7 +330,7 @@ @@ -354,6 +354,11 @@ + +